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

m.room.aliases may contain aliases from other homeservers #539

Open
richvdh opened this issue Sep 2, 2019 · 0 comments
Open

m.room.aliases may contain aliases from other homeservers #539

richvdh opened this issue Sep 2, 2019 · 0 comments
Labels
A-Client-Server Issues affecting the CS API enhancement A suggestion for a relatively simple improvement to the protocol

Comments

@richvdh
Copy link
Member

richvdh commented Sep 2, 2019

There is no validation on the aliases listed on an m.room.aliases event, so it is possible to list things which aren't valid aliases at all, or which would be valid aliases, but are on a different server:

{
  "type": "m.room.aliases",
  "state_key": "matrix.org",
  "content": {
    "aliases": [
      "random jakj # 2342 !",
      "#foo:sw1v.org"
    ]
  }

Both of these currently confuse riot-web at least.

Related: #176, #533

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Client-Server Issues affecting the CS API enhancement A suggestion for a relatively simple improvement to the protocol
Projects
None yet
Development

No branches or pull requests

2 participants