-
Notifications
You must be signed in to change notification settings - Fork 375
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.
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
Remove aliases
field from /publicRooms
#3056
Labels
spec-bug
Something which is in the spec, but is wrong
Comments
ah ha, we do have an issue for this. In a different line of investigation we ran across this exact issue. Proposed for removal here: #3624 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
... or at least make a suggestion that servers don't return it
Link to problem area:
https://spec.matrix.org/unstable/client-server-api/#get_matrixclientr0publicrooms
Issue
The spec for
/publicRooms
suggests that servers should return analiases
field from the/publicRooms
. However, MSC2432 changed most of the rest of the spec so thatm.room.aliases
events are no longer treated specially, and matrix-org/synapse#6970 removed support for this field from Synapse.Although this endpoint was never called out explicitly in MSC2432, I think that was just an oversight. It might be nice if the situation was at least made clearer in the spec.
Expected behaviour
Remove the field altogether? Or call out that it may not be returned by common server impls, and recommend that servers don't populate it?
The text was updated successfully, but these errors were encountered: