-
-
Notifications
You must be signed in to change notification settings - Fork 94
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
Consider replacing m.room.aliases
with an m.room.alias
event listing a single alias
#532
Comments
As pointed out on matrix-org/matrix-spec-proposals#1130: one problem with this is that currently, the matrix protocol doesn't support deleting state (#456). We could instead set an |
the whole |
we should also take the opportunity to harden up the validation rules to solve things like https://github.com/matrix-org/matrix-doc/issues/2277. |
|
Indeed. MSC2432 removed the special meaning for |
Currently, the
m.room.aliases
state event has a state_key corresponding to a given server name, and it lists all the aliases for a given room on that server (or at least, it is intended to). This runs the risk of races when making several changes to the directory at once.Instead, we could have an
m.room.alias
event for each alias. For example:(This was originally proposed in matrix-org/matrix-spec-proposals#1130, but I have split it to here to remove redundant content.)
The text was updated successfully, but these errors were encountered: