Skip to content

Commit

Permalink
Adding source context for the logging in RequestLoggingMiddleware. Fi…
Browse files Browse the repository at this point in the history
…xing Issue #123
  • Loading branch information
adamlith committed Aug 29, 2019
1 parent 7740fbe commit c70574c
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/Serilog.AspNetCore/AspNetCore/RequestLoggingMiddleware.cs
Original file line number Diff line number Diff line change
Expand Up @@ -70,9 +70,10 @@ public async Task Invoke(HttpContext httpContext)

bool LogCompletion(HttpContext httpContext, DiagnosticContextCollector collector, int statusCode, double elapsedMs, Exception ex)
{
var logger = Log.Logger.ForContext<RequestLoggingMiddleware>();
var level = statusCode > 499 ? LogEventLevel.Error : LogEventLevel.Information;

if (!Log.IsEnabled(level)) return false;
if (!logger.IsEnabled(level)) return false;

if (!collector.TryComplete(out var collectedProperties))
collectedProperties = NoProperties;
Expand All @@ -87,7 +88,7 @@ bool LogCompletion(HttpContext httpContext, DiagnosticContextCollector collector
});

var evt = new LogEvent(DateTimeOffset.Now, level, ex, _messageTemplate, properties);
Log.Write(evt);
logger.Write(evt);

return false;
}
Expand Down

0 comments on commit c70574c

Please sign in to comment.