Are server names supposed to be case insensitive? #790
joelverhagen
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Pre-submission Checklist
Discussion Topic
Today, the server.json
nameappears to be case sensitive. This is fine, but it does lead to some strange things:The part after the
/is free form soFoo,foo, andFOOcan all coexist.The DNS based auth allows uppercase DNS names (non-idiomatic) to be logged in with, therefore
COM.contoso/*can be used.See these entries in staging:
com.microsoft.mcp/knapcode.samplemcpserverhttps://staging.registry.modelcontextprotocol.io/v0.1/servers/com.joelverhagen.mcp%2Fknapcode.samplemcpserver/versions/latest
com.microsoft.mcp/Knapcode.SampleMcpServerhttps://staging.registry.modelcontextprotocol.io/v0.1/servers/com.joelverhagen.mcp%2FKnapcode.SampleMcpServer/versions/latest
com.microsoft.MCP/Knapcode.SampleMcpServerhttps://staging.registry.modelcontextprotocol.io/v0.1/servers/com.joelverhagen.MCP%2FKnapcode.SampleMcpServer/versions/latest
(all different latest versions because they are different
namevalues).Should this be allowed?
It seems like this should be banned either by making a single casing allowed (perhaps the first version defines the casing, but then it's unfixable), by forcing lowercase (breaking change), or by going fully case insensitive.
Thankfully only A-Z and a-z is supported (no non-latin characters) so case sensitivity is well defined (no culture or unicode issues).
Beta Was this translation helpful? Give feedback.
All reactions