Add short-circuit if not enabled

microsoft · Oct 9, 2020 · cbc128d · cbc128d
1 parent 4e381b8
commit cbc128d
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/ReverseProxy/Telemetry/DiagnosticsHandler.cs b/src/ReverseProxy/Telemetry/DiagnosticsHandler.cs
@@ -52,6 +52,12 @@ protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage
                 throw new ArgumentNullException(nameof(request)); //, SR.net_http_handler_norequest); TODO: Is this really necessary?
             }
 
+            // Short-circuit to inner handler if we're not enabled at all
+            if (!IsEnabled())
+            {
+                return await base.SendAsync(request, cancellationToken).ConfigureAwait(false);
+            }
+
             Activity activity = null;
 
             // if there is no listener, but propagation is enabled (with previous IsEnabled() check)