nit: remove request.path tracing field

shuttle-hq · Mar 12, 2024 · 0be50c3 · 0be50c3
1 parent 1edf407
commit 0be50c3
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/common/src/backends/metrics.rs b/common/src/backends/metrics.rs
@@ -197,7 +197,6 @@ macro_rules! request_span {
             http.status_code = tracing::field::Empty,
             // A bunch of extra things for metrics
             // Should be able to make this clearer once `Valuable` support lands in tracing
-            request.path = path,
             $($field)*
         )
         }
@@ -253,7 +252,6 @@ mod tests {
                 .with_span_field("http.uri")
                 .with_span_field("http.method")
                 .with_span_field("http.status_code")
-                .with_span_field("request.path")
                 .was_closed()
                 .finalize();
 
@@ -297,7 +295,6 @@ mod tests {
                 .with_span_field("http.uri")
                 .with_span_field("http.method")
                 .with_span_field("http.status_code")
-                .with_span_field("request.path")
                 .with_span_field("request.params.user_name")
                 .with_span_field("extra")
                 .was_closed()