remove url.path because it is now set from HttpTelemetryModule

open-telemetry · Jul 20, 2024 · adeb5cf · adeb5cf
1 parent bec8e52
commit adeb5cf
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
@@ -64,7 +64,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;
@@ -83,8 +82,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))