如何使 elmah.io 与 ASP.NET 核心和错误处理中间件一起工作?
How to make elmah.io work well with ASP.NET Core and error handing middleware?
注意:这个问题是关于 elmah.io (https://elmah.io/),基于云的异常日志记录服务,而不是传统的 Elmah .Net 库。
我正在使用 ASP.NET Core 并且有一个简单的异常处理中间件。
public class HandleExceptionMiddleware
{
public HandleExceptionMiddleware(RequestDelegate next)
{
Next = next;
}
RequestDelegate Next { get; }
public async Task Invoke(HttpContext httpContext)
{
try
{
await Next(httpContext);
}
catch (Exception ex)
{
await HandleExceptionAsync(httpContext, ex);
}
}
Task HandleExceptionAsync(HttpContext context, Exception ex)
{
var code = HttpStatusCode.InternalServerError;
if (ex is ArgumentException)
code = HttpStatusCode.BadRequest;
var result = JsonConvert.SerializeObject(new { message = ex.Message });
context.Response.ContentType = "application/json";
context.Response.StatusCode = (int)code;
return context.Response.WriteAsync(result);
}
}
此中间件将 return 以下 JSON 响应取决于它看到的异常:
对于 ArgumentException:
HTTP/1.1 400 Bad Request
{"message":""}
对于所有其他例外情况:
HTTP/1.1 500 Internal Server Error
{"message":""}
我希望 elmah.io 记录 500
响应并忽略 400
响应(这是默认的 elmah.io 配置)。但是,当异常处理程序和 elmah.io 在 Startup 的 Configure 挂钩中按此顺序注册时,不会记录任何内容 elmah.io:
app.UseElmahIo("API_KEY", new Guid("LOG_ID"));
app.UseMiddleware<HandleExceptionMiddleware>()
但是,如果我将注册顺序更改为以下内容,所有内容都会被记录(包括 400 条回复)。这是有道理的,因为 elmah.io 在 HandleExceptionMiddleware
有机会更改响应之前处理异常:
app.UseMiddleware<HandleExceptionMiddleware>()
app.UseElmahIo("API_KEY", new Guid("LOG_ID"));
配置这些服务以便 elmah.io 记录 500 个响应并忽略 400 个响应的最佳方法是什么?
我想出的唯一解决方法是创建并注册 2 个异常处理中间件,而不是 1 个。一个在 elmah.io 之前注册,一个在 elmah.io 之后注册。它有效,但看起来有点难看:
app.UseMiddleware<HandleInternalExceptionMiddleware>() // set 500 responses (these will have already been logged in elmah.io)
app.UseElmahIo("API_KEY", new Guid("LOG_ID"));
app.UseMiddleware<HandleExternalExceptionMiddleware>() // set 400 responses but ignore exceptions that should return 500 (these won't be logged in elmah.io)
我在这里创建了一个示例项目来演示此行为:
https://github.com/johnnyoshika/elmah-io-experiment
配置 elmah.io 的正确方法是使用行为 2。您想在调用其他处理异常的方法之后调用 UseElmahIo
方法。这是因为很多错误处理中间件(包括你的 HandleExceptionMiddleware
)吞噬了所有异常并将结果转换为其他东西。在您的情况下,HandleExceptionMiddleware
捕获所有异常并设置新的响应。在这种情况下,我们的中间件永远不会收到有关异常的通知(正如您提到的那样)。
我们有几种不同的方法来解决这个问题:
解决方案 1
在调用 UseMiddleware
之后调用 UseElmahIo
并添加自定义忽略过滤器以忽略最终成为错误请求的错误:
app.UseMiddleware<HandleExceptionMiddleware>();
app.UseElmahIo("API_KEY", new Guid("LOG_ID"), new ElmahIoSettings
{
OnFilter = msg => msg.Type == typeof(ArgumentException).Name
});
这种方法的缺点是,您需要在 HandleExceptionMiddleware
和 elmah.io 配置中维护一组相似的规则。
解决方案 2
在调用 UseMiddleware
之前调用 UseElmahIo
并指定要记录的状态代码,即使未抛出异常(在本例中被 HandleExceptionMiddleware
吞没):
app.UseElmahIo("API_KEY", new Guid("LOG_ID"), new ElmahIoSettings
{
HandledStatusCodesToLog = new List<int> { 404, 500, ... }
});
app.UseMiddleware<HandleExceptionMiddleware>();
这种方法的缺点是,您需要手动指定所有状态代码,并且来自实际抛出的异常的信息在 elmah.io 上不可用。再次这样做的原因是,HandleExceptionMiddleware
使 elmah.io 无法看到抛出的异常。
我个人更喜欢解决方案 1,因为它可以确保捕获所有异常,包括堆栈跟踪和异常类型等信息。
注意:这个问题是关于 elmah.io (https://elmah.io/),基于云的异常日志记录服务,而不是传统的 Elmah .Net 库。
我正在使用 ASP.NET Core 并且有一个简单的异常处理中间件。
public class HandleExceptionMiddleware
{
public HandleExceptionMiddleware(RequestDelegate next)
{
Next = next;
}
RequestDelegate Next { get; }
public async Task Invoke(HttpContext httpContext)
{
try
{
await Next(httpContext);
}
catch (Exception ex)
{
await HandleExceptionAsync(httpContext, ex);
}
}
Task HandleExceptionAsync(HttpContext context, Exception ex)
{
var code = HttpStatusCode.InternalServerError;
if (ex is ArgumentException)
code = HttpStatusCode.BadRequest;
var result = JsonConvert.SerializeObject(new { message = ex.Message });
context.Response.ContentType = "application/json";
context.Response.StatusCode = (int)code;
return context.Response.WriteAsync(result);
}
}
此中间件将 return 以下 JSON 响应取决于它看到的异常:
对于 ArgumentException:
HTTP/1.1 400 Bad Request
{"message":""}
对于所有其他例外情况:
HTTP/1.1 500 Internal Server Error
{"message":""}
我希望 elmah.io 记录 500
响应并忽略 400
响应(这是默认的 elmah.io 配置)。但是,当异常处理程序和 elmah.io 在 Startup 的 Configure 挂钩中按此顺序注册时,不会记录任何内容 elmah.io:
app.UseElmahIo("API_KEY", new Guid("LOG_ID"));
app.UseMiddleware<HandleExceptionMiddleware>()
但是,如果我将注册顺序更改为以下内容,所有内容都会被记录(包括 400 条回复)。这是有道理的,因为 elmah.io 在 HandleExceptionMiddleware
有机会更改响应之前处理异常:
app.UseMiddleware<HandleExceptionMiddleware>()
app.UseElmahIo("API_KEY", new Guid("LOG_ID"));
配置这些服务以便 elmah.io 记录 500 个响应并忽略 400 个响应的最佳方法是什么?
我想出的唯一解决方法是创建并注册 2 个异常处理中间件,而不是 1 个。一个在 elmah.io 之前注册,一个在 elmah.io 之后注册。它有效,但看起来有点难看:
app.UseMiddleware<HandleInternalExceptionMiddleware>() // set 500 responses (these will have already been logged in elmah.io)
app.UseElmahIo("API_KEY", new Guid("LOG_ID"));
app.UseMiddleware<HandleExternalExceptionMiddleware>() // set 400 responses but ignore exceptions that should return 500 (these won't be logged in elmah.io)
我在这里创建了一个示例项目来演示此行为: https://github.com/johnnyoshika/elmah-io-experiment
配置 elmah.io 的正确方法是使用行为 2。您想在调用其他处理异常的方法之后调用 UseElmahIo
方法。这是因为很多错误处理中间件(包括你的 HandleExceptionMiddleware
)吞噬了所有异常并将结果转换为其他东西。在您的情况下,HandleExceptionMiddleware
捕获所有异常并设置新的响应。在这种情况下,我们的中间件永远不会收到有关异常的通知(正如您提到的那样)。
我们有几种不同的方法来解决这个问题:
解决方案 1
在调用 UseMiddleware
之后调用 UseElmahIo
并添加自定义忽略过滤器以忽略最终成为错误请求的错误:
app.UseMiddleware<HandleExceptionMiddleware>();
app.UseElmahIo("API_KEY", new Guid("LOG_ID"), new ElmahIoSettings
{
OnFilter = msg => msg.Type == typeof(ArgumentException).Name
});
这种方法的缺点是,您需要在 HandleExceptionMiddleware
和 elmah.io 配置中维护一组相似的规则。
解决方案 2
在调用 UseMiddleware
之前调用 UseElmahIo
并指定要记录的状态代码,即使未抛出异常(在本例中被 HandleExceptionMiddleware
吞没):
app.UseElmahIo("API_KEY", new Guid("LOG_ID"), new ElmahIoSettings
{
HandledStatusCodesToLog = new List<int> { 404, 500, ... }
});
app.UseMiddleware<HandleExceptionMiddleware>();
这种方法的缺点是,您需要手动指定所有状态代码,并且来自实际抛出的异常的信息在 elmah.io 上不可用。再次这样做的原因是,HandleExceptionMiddleware
使 elmah.io 无法看到抛出的异常。
我个人更喜欢解决方案 1,因为它可以确保捕获所有异常,包括堆栈跟踪和异常类型等信息。