Improved re-entrancy Scope Lock for Session Value Layout Render

src/Shared/Internal/RendererReEntrantManager.cs

@@ -23,10 +23,13 @@ internal RendererReEntrantManager(HttpContextBase context)
         {
             _httpContext = context;
             _obtainedLock = false;
+            TryGetLock();
         }
 
         private readonly HttpContextBase _httpContext;
 
+        internal bool IsLockAcquired => _obtainedLock;
+
 #if NET35
         private static readonly object ReEntrantLock = new object();
 
@@ -68,19 +71,17 @@ private void Unlock()
         private bool _obtainedLock;
 
 
-        internal bool TryGetLock()
+        private void TryGetLock()
         {
             // If already locked, return false
             if (IsLocked())
             {
-                return false;
+                return;
             }
             // Get the lock
             Lock();
             // Mark that we locked it, not another instance locked it
             _obtainedLock = true;
-            // Return to the caller that we locked it
-            return true;
         }
 
         private void DisposalImpl()

src/Shared/LayoutRenderers/AspNetSessionValueLayoutRenderer.cs

@@ -102,7 +102,7 @@ protected override void DoAppend(StringBuilder builder, LogEventInfo logEvent)
 
             using (var reEntry = new RendererReEntrantManager(context))
             {
-                if (!reEntry.TryGetLock())
+                if (!reEntry.IsLockAcquired)
                 {
                     InternalLogger.Error(
                         $"Reentrant log event detected. Logging when inside the scope of another log event can cause a StackOverflowException. LogEventInfo.Message:{logEvent.Message}");