Ignore empty names

dgageot · dgageot · commit ca7c5798935f · 2025-07-27T11:09:53.000+02:00
Signed-off-by: David Gageot &lt;david.gageot@docker.com&gt;
diff --git a/cmd/docker-mcp/internal/gateway/transport.go b/cmd/docker-mcp/internal/gateway/transport.go
@@ -100,7 +100,12 @@ func (g *Gateway) startCentralStreamingServer(ctx context.Context, newMCPServer
 func parseServerNames(serverNames string) []string {
 	var names []string
 	for name := range strings.SplitSeq(serverNames, ",") {
-		names = append(names, strings.TrimSpace(name))
+		name := strings.TrimSpace(name)
+		if name == "" {
+			continue
+		}
+
+		names = append(names, name)
 	}
 	return names
 }

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,12 @@ func (g *Gateway) startCentralStreamingServer(ctx context.Context, newMCPServer`
`100`	`100`	`func parseServerNames(serverNames string) []string {`
`101`	`101`	`var names []string`
`102`	`102`	`for name := range strings.SplitSeq(serverNames, ",") {`
`103`		`- names = append(names, strings.TrimSpace(name))`
	`103`	`+ name := strings.TrimSpace(name)`
	`104`	`+ if name == "" {`
	`105`	`+ continue`
	`106`	`+ }`
	`107`	`+`
	`108`	`+ names = append(names, name)`
`104`	`109`	`}`
`105`	`110`	`return names`
`106`	`111`	`}`