Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

sometimes we use an out-of-date server for send_joins #2418

Closed
richvdh opened this issue Aug 17, 2017 · 4 comments
Closed

sometimes we use an out-of-date server for send_joins #2418

richvdh opened this issue Aug 17, 2017 · 4 comments

Comments

@richvdh
Copy link
Member

richvdh commented Aug 17, 2017

... which in turn means that the joining user gets a very old view of the room state

@richvdh
Copy link
Member Author

richvdh commented Aug 17, 2017

For example, when one tries to join #megolm:matrix.org, matrix.org returns the following list of servers in the room:

  • dmp-develop.com
  • matrix.freelock.com
  • darmstadt.ccc.de
  • jki.re
  • onedefence.com
  • terracrypt.net
  • sw1v.org
  • matrix.org
  • exul.org
  • t2l.io

Of these, dmp-develop.com is offline. I don't know why we don't use matrix.freelock.com. Anyway, I've seen various servers use darmstadt.ccc.de. But that server left a long time ago. So the user gets an ancient state and timeline snapshot, and it takes ages for backfilling etc to sort it all out.

@richvdh
Copy link
Member Author

richvdh commented Aug 17, 2017

Of course if the user sends any encrypted messages while the server is trying to catch up, then it will exclude any users who joined the room since darmstadt.ccc.de left.

@richvdh
Copy link
Member Author

richvdh commented Sep 6, 2018

Another example: if you try to join #riot via hveem.no, you get a room consisting entirely of a single 'create' event. :/

@richvdh richvdh changed the title sometimes we use an out-of-date server for make_joins sometimes we use an out-of-date server for send_joins Sep 6, 2018
richvdh added a commit that referenced this issue Sep 18, 2018
When we do a join, always try the server we used for the alias lookup first.

Fixes #2418
richvdh added a commit that referenced this issue Sep 18, 2018
When we do a join, always try the server we used for the alias lookup first.

Fixes #2418
@richvdh
Copy link
Member Author

richvdh commented Sep 18, 2018

Fixed by #3899

@richvdh richvdh closed this as completed Sep 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant