feature: Shorten the operationname of HttpClient ExistSpan (#354)

snakorse · 郭刚平 · web-flow · commit 78f16bc6a669 · 2020-11-24T16:47:23.000+08:00
Co-authored-by: 郭刚平 &lt;gangping.guo@xiaobao100.com&gt;
diff --git a/src/SkyApm.Diagnostics.HttpClient/Handlers/DefaultRequestDiagnosticHandler.cs b/src/SkyApm.Diagnostics.HttpClient/Handlers/DefaultRequestDiagnosticHandler.cs
@@ -16,6 +16,7 @@
  *
  */
 
+using System;
 using System.Linq;
 using System.Net.Http;
 using SkyApm.Common;
@@ -43,7 +44,7 @@ public bool OnlyMatch(HttpRequestMessage request)
 
         public void Handle(ITracingContext tracingContext, HttpRequestMessage request)
         {
-            var operationName = request.RequestUri.ToString();
+            var operationName = request.RequestUri.GetLeftPart(UriPartial.Path);
             var shouldStopPropagation = _httpClientDiagnosticConfig.StopHeaderPropagationPaths != null 
                 && _httpClientDiagnosticConfig.StopHeaderPropagationPaths
                     .Any(pattern => FastPathMatcher.Match(pattern, operationName));

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@`
`16`	`16`	`*`
`17`	`17`	`*/`
`18`	`18`
	`19`	`+using System;`
`19`	`20`	`using System.Linq;`
`20`	`21`	`using System.Net.Http;`
`21`	`22`	`using SkyApm.Common;`
`@@ -43,7 +44,7 @@ public bool OnlyMatch(HttpRequestMessage request)`
`43`	`44`
`44`	`45`	`public void Handle(ITracingContext tracingContext, HttpRequestMessage request)`
`45`	`46`	`{`
`46`		`- var operationName = request.RequestUri.ToString();`
	`47`	`+ var operationName = request.RequestUri.GetLeftPart(UriPartial.Path);`
`47`	`48`	`var shouldStopPropagation = _httpClientDiagnosticConfig.StopHeaderPropagationPaths != null`
`48`	`49`	`&& _httpClientDiagnosticConfig.StopHeaderPropagationPaths`
`49`	`50`	`.Any(pattern => FastPathMatcher.Match(pattern, operationName));`