aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBond_009 <bond.009@outlook.com>2022-11-30 22:07:34 +0100
committerBond_009 <bond.009@outlook.com>2022-11-30 22:11:47 +0100
commitb2ce70987c2f724436514e8f91f9f455be7d13ff (patch)
tree98f8e33ef71865e963bdcc489436494147f12d26
parentfea01a1ccb44dc995c6034ebdc72f0b75c86cfb7 (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.cs6
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;
});