Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Registering a 3pid can cause you to be kicked out of a room #7838

Open
richvdh opened this issue Jul 13, 2020 · 1 comment
Open

Registering a 3pid can cause you to be kicked out of a room #7838

richvdh opened this issue Jul 13, 2020 · 1 comment
Labels
A-3PID 3rd party identifiers: e.g. email, phone number O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Major functionality / product severely impaired, no satisfactory workaround. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. z-WTF Causing the user to exclaim! These issues are high impact and low effort.

Comments

@richvdh
Copy link
Member

richvdh commented Jul 13, 2020

And worse, if you were the only one in the room, you will be unable to rejoin.

Steps to repeat:

  1. Be the only user in a room.
  2. Send a 3pid invite to the room, to an unregistered 3pid.
  3. Register the 3pid yourself.
  4. Observe that you are no longer a member of the room; rather you have an invite to that room which you cannot accept ("no known servers", or, if the room's extremities get cleared by delete_old_current_state_events, "No create event in auth_events").

Until #7804 landed, you wouldn't have been able to reject it, either. That much at least seems to work now.

@richvdh
Copy link
Member Author

richvdh commented Jul 14, 2020

Until #7804 landed, you wouldn't have been able to reject it, either. That much at least seems to work now.

actually I suspect this is incorrect. As long as the room hasn't been cleared out by delete_old_current_state_events, you can still reject the invite. If the room is cleared out by delete_old_current_state_events, you get a stuck invite, as per #7139.

@anoadragon453 anoadragon453 added z-bug (Deprecated Label) p1 labels Jul 20, 2020
@neilisfragile neilisfragile added z-p2 (Deprecated Label) and removed p1 labels Jul 29, 2020
@MadLittleMods MadLittleMods added T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. A-3PID 3rd party identifiers: e.g. email, phone number labels May 3, 2022
@DMRobertson DMRobertson added S-Major Major functionality / product severely impaired, no satisfactory workaround. z-WTF Causing the user to exclaim! These issues are high impact and low effort. O-Uncommon Most users are unlikely to come across this or unexpected workflow and removed z-bug (Deprecated Label) z-p2 (Deprecated Label) labels Nov 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-3PID 3rd party identifiers: e.g. email, phone number O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Major functionality / product severely impaired, no satisfactory workaround. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. z-WTF Causing the user to exclaim! These issues are high impact and low effort.
Projects
None yet
Development

No branches or pull requests

5 participants