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

Transform automatically non allowed character in Nickname ( Matrix -> XMPP ) #311

Open
JokerGermany opened this issue Apr 11, 2022 · 1 comment
Labels
S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.

Comments

@JokerGermany
Copy link

JokerGermany commented Apr 11, 2022

Describe the bug
When a Nickname of Matrix user contains in XMPP not allowed character like emoji, they should be converted automatically to prevent that other xmpp users can't read his messages.

To Reproduce

  1. Change your Global Nick of a matrix account to something with a emoji in it
  2. join a XMPP Room
  3. Send a message
  4. Some XMPP User won't be able to read the messages.

Expected behavior
Variant 1:
The bridge changed the nickname of the Person automatically.

Variant 2:
The bridge messages the user and tell him, that he need to change his nickname otherwise not every XMPP User will be able to read his messages.

Additional context
For Variant 2:
@maranda implemented that the bridge forward changing the roomnick:
maranda@6a0743b
maranda@86b5fbf
maranda@dfdc29f
maranda@49a15c5
If this is not implemented, the Global Name has to be changed!

@JokerGermany JokerGermany changed the title Transform automatically non allowed character in der Nickname ( Matrix -> XMPP ) Transform automatically non allowed character in Nickname ( Matrix -> XMPP ) Apr 11, 2022
@Half-Shot
Copy link
Collaborator

Half-Shot commented May 18, 2022

I suspect we want to do variant 1. I actually thought we did this already! Calling this out as a T-major since it's hard to easily workaround.

@Half-Shot Half-Shot added T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems. S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.
Projects
None yet
Development

No branches or pull requests

2 participants