Make span name without query string (#1013)

Query strings shouldn't be included in span names, as this prevents
tracing consumers from properly aggregating spans, etc.

This change modifies span names to only include URI paths.

Signed-off-by: aatarasoff <[email protected]>

Author: aatarasoff

linkerd/trace-context/src/service.rs

@@ -96,11 +96,7 @@ where
                     let start = SystemTime::now();
                     let req_labels = Self::request_labels(&req);
                     let mut sink = self.sink.clone();
-                    let span_name = req
-                        .uri()
-                        .path_and_query()
-                        .map(|pq| pq.as_str().to_owned())
-                        .unwrap_or_default();
+                    let span_name = req.uri().path().to_owned();
                     return Either::Right(Box::pin(self.inner.call(req).map_ok(move |rsp| {
                         // Emit the completed span with the response metadata.
                         let span = Span {