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.

Sign up for GitHub

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

Jump to bottom

Fails to show user ID with slash #2617

Open
shadow-identity opened this issue Mar 28, 2024 · 4 comments
Open

Fails to show user ID with slash #2617

shadow-identity opened this issue Mar 28, 2024 · 4 comments
Labels
A-Event Rendering How events are shown in the timeline A-Rich-Text-Editor O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@shadow-identity
Copy link

Steps to reproduce

  1. Open a message containing user mention. User ID should have slash "/" in it, for example @s/ash:matrix.org.
  2. Tap on user ID

Outcome

What did you expect?

To see the user info, like with any other user.

What happened instead?

It opens https://matrix.to/#/@it/sme:matrix.org. Matrix.to fails to show the user (incorrectly reporting "wrong URL"). So user can not see any user info.

Please note that the slash in the user id @it/sme:matrix.org is not escaped.
But even escaped slash URL matrix.to returns the error.

Your phone model

No response

Operating system version

Android 14

Application version and app store

Element X 0.4.6

Homeserver

No response

Will you send logs?

No

Are you willing to provide a PR?

No
@shadow-identity shadow-identity added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Mar 28, 2024
@shadow-identity
Copy link
Author

BTW same happens at least in Element Web and Element for macOS

@bmarty
Copy link
Member

bmarty commented Mar 28, 2024

Thanks for reporting.

/ is accepted in the Matrix userId localpart grammar: https://spec.matrix.org/latest/appendices/#user-identifiers, so this will have to be fixed at some point.

@bmarty bmarty added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Uncommon Most users are unlikely to come across this or unexpected workflow A-Rich-Text-Editor A-Event Rendering How events are shown in the timeline labels Mar 28, 2024
@bmarty
Copy link
Member

bmarty commented Mar 28, 2024

For the issue with matrix.to not working properly, you may want to report it in this project https://github.com/matrix-org/matrix.to. Thanks!

@shadow-identity
Copy link
Author

Thanks, I found that they already have it: matrix-org/matrix.to#288

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Event Rendering How events are shown in the timeline A-Rich-Text-Editor O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@shadow-identity @bmarty