Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Server names are case sensitive in /publicRooms #1619

Closed
timokoesters opened this issue Aug 18, 2023 · 0 comments · Fixed by #1638
Closed

Server names are case sensitive in /publicRooms #1619

timokoesters opened this issue Aug 18, 2023 · 0 comments · Fixed by #1638
Labels
clarification An area where the expected behaviour is understood, but the spec could do with being more explicit

Comments

@timokoesters
Copy link

The spec should mention that the server field of GET/PUT /publicRooms is case sensitive. https://spec.matrix.org/v1.7/client-server-api/#get_matrixclientv3publicrooms
The client should lowercase it before sending. This probably effects other endpoints too.

Alternatively, servers can lowercase it or the entire spec can be overhauled to make server names case insensitive everywhere.
@timokoesters timokoesters added the clarification An area where the expected behaviour is understood, but the spec could do with being more explicit label Aug 18, 2023
@mohit1523 mohit1523 mentioned this issue Sep 14, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clarification An area where the expected behaviour is understood, but the spec could do with being more explicit
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add "case sensitive" to /publicRooms
1 participant
@timokoesters