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.

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

Room aliases list doesn't take into account multiple m.room.aliases states #106

Open
vurpo opened this issue Mar 19, 2017 · 0 comments
Open

Comments

@vurpo
Copy link

vurpo commented Mar 19, 2017

The Room.update_aliases method does not take into account that there can be multiple m.room.aliases states. Because room aliases from separate HSes are in separate m.room.aliases states (with the HS name as the state key), this means the update_aliases function will actually set the aliases list to only contain the aliases from one HS, disregarding the others.

Example (the code should list all aliases for Matrix HQ):

>>> client.rooms['!cURbafjkfsMDVwdRDQ:matrix.org'].update_aliases()
True
>>> client.rooms['!cURbafjkfsMDVwdRDQ:matrix.org'].aliases
['#matrix:utzutzutz.net']
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants