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

Behaviour when trying to DM a federated user on a server with federation disabled is awful #27438

Closed
ara4n opened this issue May 7, 2024 · 1 comment
Labels
A-DM-Start Creating a DM with another user A-Federation S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented May 7, 2024

Steps to reproduce

  1. Logged into the OIDC test lab
  2. Tried to DM myself on matrix.org (see also Prompt to set up E2EE on new account didn't happen as expected with OIDC #27437)
  3. Got an ambiguous "failed" error in the proto DM.
  4. Tried again, which created a new proto DM.
  5. Restarted the app
  6. Now have two proto-dms in the room list, which provide a 'join' button.
  7. Hitting the join button produces a fugly error complaining about trying to call /join on a local+... room ID.

Outcome

What did you expect?

  • Reasons why I can't send messages should be shown to the user in the nice red warning error, so the user knows what's going on.
  • Failed proto-DMs should be reused when trying to contact the same user
  • Failed proto-DMs should not switch to showing a 'join' button which doesn't work, when you refresh

What happened instead?

  • The error from the server ({errcode: "M_FORBIDDEN", error: "Federation denied with matrix.org."}) was swallowed entirely by the UI.
  • I ended up with multiple broken DMs to the same user
  • They ended up giving me useless buggy 'join' buttons

Operating system

No response

Browser information

No response

URL for webapp

https://pr12370--matrix-react-sdk.netlify.app/

Application version

No response

Homeserver

No response

Will you send logs?

No

@ara4n ara4n added the T-Defect label May 7, 2024
@dosubot dosubot bot added A-DM-Start Creating a DM with another user A-Federation S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels May 7, 2024
@t3chguy
Copy link
Member

t3chguy commented May 7, 2024

Duplicate of #24991

@t3chguy t3chguy marked this as a duplicate of #24991 May 7, 2024
@t3chguy t3chguy closed this as not planned Won't fix, can't repro, duplicate, stale May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-DM-Start Creating a DM with another user A-Federation S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants