AppMetrics#588 fixing apdex not measure in tracking middleware due to…

… incorrect condition on check for websocket connection
seif · Jul 3, 2020 · 2b80de9 · 2b80de9
1 parent eaccf2f
commit 2b80de9
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/AspNetCore/src/App.Metrics.AspNetCore.Tracking/Middleware/ApdexMiddleware.cs b/src/AspNetCore/src/App.Metrics.AspNetCore.Tracking/Middleware/ApdexMiddleware.cs
@@ -37,12 +37,9 @@ public ApdexMiddleware(
         public Task Invoke(HttpContext context)
             // ReSharper restore UnusedMember.Global
         {
-            if (!context.WebSockets.IsWebSocketRequest)
-            {
-                return _next(context);
-            }
-
-            return MeasureTransaction(context);
+            return context.WebSockets.IsWebSocketRequest 
+                ? _next(context)
+                : MeasureTransaction(context);
         }
 
         private async Task MeasureTransaction(HttpContext context)