Make scheme comparisons case-insensitive in IsDefaultPort

Copilot · halter73 · Copilot · commit fe74a82b6fc6 · 2026-02-13T02:49:46.000Z
Use StringComparison.OrdinalIgnoreCase for scheme comparisons to handle HTTP/HTTPS in any case.

Co-authored-by: halter73 &lt;54385+halter73@users.noreply.github.com&gt;
diff --git a/src/ModelContextProtocol.AspNetCore/Authentication/McpAuthenticationHandler.cs b/src/ModelContextProtocol.AspNetCore/Authentication/McpAuthenticationHandler.cs
@@ -77,7 +77,8 @@ private async Task<bool> HandleDefaultResourceMetadataRequestAsync()
 
     private static bool IsDefaultPort(string scheme, int port)
     {
-        return (scheme == "http" && port == 80) || (scheme == "https" && port == 443);
+        return (scheme.Equals("http", StringComparison.OrdinalIgnoreCase) && port == 80) || 
+               (scheme.Equals("https", StringComparison.OrdinalIgnoreCase) && port == 443);
     }
 
     /// <summary>

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,8 @@ private async Task<bool> HandleDefaultResourceMetadataRequestAsync()`
`77`	`77`
`78`	`78`	`private static bool IsDefaultPort(string scheme, int port)`
`79`	`79`	`{`
`80`		`- return (scheme == "http" && port == 80) \|\| (scheme == "https" && port == 443);`
	`80`	`+ return (scheme.Equals("http", StringComparison.OrdinalIgnoreCase) && port == 80) \|\|`
	`81`	`+ (scheme.Equals("https", StringComparison.OrdinalIgnoreCase) && port == 443);`
`81`	`82`	`}`
`82`	`83`
`83`	`84`	`/// <summary>`