remove url.path because it is now set from HttpTelemetryModule

open-telemetry · Jun 13, 2024 · 39ccd68 · 39ccd68
1 parent 0d797a6
commit 39ccd68
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/src/OpenTelemetry.Instrumentation.AspNet/Implementation/HttpInListener.cs b/src/OpenTelemetry.Instrumentation.AspNet/Implementation/HttpInListener.cs
@@ -65,7 +65,6 @@ private void OnStartActivity(Activity activity, HttpContext context)
             }
 
             var request = context.Request;
-            var requestValues = request.Unvalidated;
 
             // see the spec https://github.com/open-telemetry/semantic-conventions/blob/v1.24.0/docs/http/http-spans.md
             var originalHttpMethod = request.HttpMethod;
@@ -84,8 +83,6 @@ private void OnStartActivity(Activity activity, HttpContext context)
                 activity.SetTag(SemanticConventions.AttributeNetworkProtocolVersion, protocolVersion);
             }
 
-            activity.SetTag(SemanticConventions.AttributeUrlPath, requestValues.Path);
-
             // TODO url.query should be sanitized
             var query = url.Query;
             if (!string.IsNullOrEmpty(query))