Merge pull request #375 from benaadams/log-scope-only-enabled

Proposal: Start log scope only if logging is enabled
2015-09-30 03:09:31 -07:00 · 2015-09-30 03:09:31 -07:00 · e323a14b43
parent 3d5f70fd01 4eba490217
commit e323a14b43
1 changed files with 3 additions and 1 deletions
--- a/src/Microsoft.AspNet.Hosting/Internal/HostingEngine.cs
+++ b/src/Microsoft.AspNet.Hosting/Internal/HostingEngine.cs
@ -105,7 +105,9 @@ namespace Microsoft.AspNet.Hosting.Internal

                    try
                    {
-                        using (logger.BeginScope("Request Id: {RequestId}", requestIdentifier))
+                        using (logger.IsEnabled(LogLevel.Critical)
+                            ? logger.BeginScope("Request Id: {RequestId}", requestIdentifier) 
+                            : null)
                        {
                            await application(httpContext);
                        }