Optimize the gRPC server address handling logic to support load balancing schemes such as dns:// and static:// (#615)

ElderJames · web-flow · commit 6a7ab51195dd · 2025-10-24T14:39:03.000+08:00
diff --git a/src/SkyApm.Abstractions/Config/GrpcConfig.cs b/src/SkyApm.Abstractions/Config/GrpcConfig.cs
@@ -42,8 +42,9 @@ public static string[] GetServers(this GrpcConfig config)
             var servers = config.Servers.Split(',').ToArray();
             for (int i = 0; i < servers.Length; i++)
             {
-                if (!servers[i].StartsWith("http://", StringComparison.InvariantCultureIgnoreCase)
-                        && !servers[i].StartsWith("https://", StringComparison.InvariantCultureIgnoreCase))
+                // Support gRPC load balancing schemes like dns://, static:// etc.
+                // Only add http:// prefix if no scheme is specified (no :// present)
+                if (!servers[i].Contains("://"))
                 {
                     servers[i] = $"http://{servers[i]}";
                 }

Original file line number	Diff line number	Diff line change
`@@ -42,8 +42,9 @@ public static string[] GetServers(this GrpcConfig config)`
`42`	`42`	`var servers = config.Servers.Split(',').ToArray();`
`43`	`43`	`for (int i = 0; i < servers.Length; i++)`
`44`	`44`	`{`
`45`		`- if (!servers[i].StartsWith("http://", StringComparison.InvariantCultureIgnoreCase)`
`46`		`- && !servers[i].StartsWith("https://", StringComparison.InvariantCultureIgnoreCase))`
	`45`	`+ // Support gRPC load balancing schemes like dns://, static:// etc.`
	`46`	`+ // Only add http:// prefix if no scheme is specified (no :// present)`
	`47`	`+ if (!servers[i].Contains("://"))`
`47`	`48`	`{`
`48`	`49`	`servers[i] = $"http://{servers[i]}";`
`49`	`50`	`}`