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

MXID disambiguation stays visible indefinitely long #4795

Open
rkfg opened this issue Aug 15, 2017 · 4 comments
Open

MXID disambiguation stays visible indefinitely long #4795

rkfg opened this issue Aug 15, 2017 · 4 comments
Labels
A-Disambiguation P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@rkfg
Copy link
Contributor

rkfg commented Aug 15, 2017

Description

If a user sets a name that matches other user's name the MXID disambiguation part is appended to it. However, if the other user leaves or changes their name the disambiguation part isn't getting away. That could pose problems, for instance, if a user tried Matrix first using matrix.org and then switched to their own homeserver using the same nickname. It's hard to get rid of that (@username:homeserver.org) part for everyone. It's not even consistent, some users/accounts see it and some don't: I don't see it from the work account and it's present when I log in from home using another account.

Steps to reproduce

  • join a room using a matrix.org account with username test
  • join the same room using another homeserver (say homeserver.org) and also rename to test
  • the second user is now visible as test (@username:homeserver.org)
  • switch to the first user and leave the room

The second user will still be test (@username:homeserver.org) even after refreshing the page/restarting the client. I expect the disambiguation to be resolved automatically so there are no MXID parts appended anywhere or there should be a way to refresh the data manually. Also, I think MXID should be appended to both ambiguous users, not just to the latter.

Version information

  • Platform: web/desktop (happens in both)
  • Browser: Chrome 61.0.3163.39
  • OS: Debian testing amd64
  • URL: riot.im/develop
@t3chguy
Copy link
Member

t3chguy commented Aug 15, 2017

It is appended to all ambiguous users:

image

@rkfg
Copy link
Contributor Author

rkfg commented Aug 15, 2017

Hmm? Weird, I renamed from rkfg2 to rkfg [work] and this is what it looks like at riot.im/develop:
2017-08-15_14-45-37
Same after refreshing the tab.

@lampholder lampholder added T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist P2 labels Aug 16, 2017
@uhoreg
Copy link
Member

uhoreg commented Oct 24, 2017

possible related: #2926, #468

@HarHarLinks
Copy link

  1. invite Alice @alice:example.org
  2. invite Alice @bob:example.org
  3. disinvite Alice @alice:example.org
  4. Disambiguation is shown for @bob:example.org even though there is no ambiguous member anymore

you could argue that this makes sense so you don't confuse them, but then element should do disambiguation based on all historic display names

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Disambiguation P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

5 participants