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

XMPP -> Matrix doesn't work when the MXID has capital letter #303

Open
Mikaela opened this issue Jan 9, 2022 · 5 comments
Open

XMPP -> Matrix doesn't work when the MXID has capital letter #303

Mikaela opened this issue Jan 9, 2022 · 5 comments
Labels
S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.

Comments

@Mikaela
Copy link

Mikaela commented Jan 9, 2022

Describe the bug

Matrix users whose MXID contains a capital letter (from time before that became disallowed) cannot be messaged from XMPP as according to Maranda:

You can't have caps in a JID that's against the prep profile so XMPP servers will normalize that

To Reproduce

  1. Send a message from XMPP to a Matrix user e.g. Ciblia_matrix.org@matrix.org
  2. The message never arrives.

Expected behavior

The messages go through and I can read them at Matrix.
@maranda
Copy link

maranda commented Jan 9, 2022

Just for clarity: "you can't have caps in the node portion of a JID"

@Mikaela
Copy link
Author

Mikaela commented Feb 11, 2022 •
edited
Loading

I guess this is an instance of matrix-org/matrix-spec-proposals#3794

@JokerGermany
Copy link

This is related to #281

@Half-Shot
Copy link
Collaborator

We'll probably have to solve this by QP encoding localparts (which will be ugly but functional).

@Half-Shot Half-Shot added T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems. S-Minor Impairs non-critical functionality or suitable workarounds exist labels May 18, 2022
@Half-Shot
Copy link
Collaborator

This is a S-Minor because it technically affects only users who are not spec compliant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@maranda @Mikaela @Half-Shot @JokerGermany