diff options
| author | Bond_009 <bond.009@outlook.com> | 2022-11-30 22:07:34 +0100 |
|---|---|---|
| committer | Bond_009 <bond.009@outlook.com> | 2022-11-30 22:11:47 +0100 |
| commit | b2ce70987c2f724436514e8f91f9f455be7d13ff (patch) | |
| tree | 98f8e33ef71865e963bdcc489436494147f12d26 | |
| parent | fea01a1ccb44dc995c6034ebdc72f0b75c86cfb7 (diff) | |
Change log level for slow HTTP responses from WRN TO DBG
The added log level check is there because Request.GetDisplayUrl() is a
pretty expensive call, creating a StringBuilder and string which doesn't
need to happen on most installs where debug logging is disabled
| -rw-r--r-- | Jellyfin.Server/Middleware/ResponseTimeMiddleware.cs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Jellyfin.Server/Middleware/ResponseTimeMiddleware.cs b/Jellyfin.Server/Middleware/ResponseTimeMiddleware.cs index 1c25696cd..65b64da4f 100644 --- a/Jellyfin.Server/Middleware/ResponseTimeMiddleware.cs +++ b/Jellyfin.Server/Middleware/ResponseTimeMiddleware.cs @@ -47,9 +47,10 @@ namespace Jellyfin.Server.Middleware context.Response.OnStarting(() => { watch.Stop(); - if (enableWarning && watch.ElapsedMilliseconds > warningThreshold) + var responseTimeForCompleteRequest = watch.ElapsedMilliseconds; + if (enableWarning && responseTimeForCompleteRequest > warningThreshold && _logger.IsEnabled(LogLevel.Debug)) { - _logger.LogWarning( + _logger.LogDebug( "Slow HTTP Response from {Url} to {RemoteIp} in {Elapsed:g} with Status Code {StatusCode}", context.Request.GetDisplayUrl(), context.GetNormalizedRemoteIp(), @@ -57,7 +58,6 @@ namespace Jellyfin.Server.Middleware context.Response.StatusCode); } - var responseTimeForCompleteRequest = watch.ElapsedMilliseconds; context.Response.Headers[ResponseHeaderResponseTime] = responseTimeForCompleteRequest.ToString(CultureInfo.InvariantCulture); return Task.CompletedTask; }); |
