Improve version validation with whitespace check

Copilot · lpcox · Copilot · commit 5786b3c81146 · 2026-02-02T14:56:28.000Z
- Use strings.TrimSpace in SetClientGatewayVersion to prevent whitespace-only versions
- Addresses code review feedback

Co-authored-by: lpcox &lt;15877973+lpcox@users.noreply.github.com&gt;
diff --git a/internal/mcp/connection.go b/internal/mcp/connection.go
@@ -27,7 +27,7 @@ var gatewayVersion = "dev"
 
 // SetClientGatewayVersion sets the gateway version for MCP client implementation reporting
 func SetClientGatewayVersion(version string) {
-	if version != "" {
+	if strings.TrimSpace(version) != "" {
 		gatewayVersion = version
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ var gatewayVersion = "dev"`
`27`	`27`
`28`	`28`	`// SetClientGatewayVersion sets the gateway version for MCP client implementation reporting`
`29`	`29`	`func SetClientGatewayVersion(version string) {`
`30`		`- if version != "" {`
	`30`	`+ if strings.TrimSpace(version) != "" {`
`31`	`31`	`gatewayVersion = version`
`32`	`32`	`}`
`33`	`33`	`}`