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

Missing knock->join transition on membership transition graph and table #2053

Open
richvdh opened this issue Jan 14, 2025 · 0 comments
Open
Labels
spec-bug Something which is in the spec, but is wrong

Comments

@richvdh
Copy link
Member

richvdh commented Jan 14, 2025

Link to problem area:

https://spec.matrix.org/v1.13/client-server-api/#room-membership, https://spec.matrix.org/v1.13/client-server-api/#mroommember

Issue

According to auth rules 4.3.5.3 and 4.3.6 it is legit to transition from knock to join if either (a) the join rule is restricted or knock_restricted and the join request satisfies the restrictions, or (b) the join rule is now public. (In other words, the same as a transition from leave to join.)

This transition is missing from the membership transition graph at https://spec.matrix.org/v1.13/client-server-api/#room-membership; furthermore the table at https://spec.matrix.org/v1.13/client-server-api/#mroommember states "Must never happen" which is flat out wrong.

Expected behaviour

Update the knock -> join transition to be the same as the leave -> join transition.

@richvdh richvdh added the spec-bug Something which is in the spec, but is wrong label Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec-bug Something which is in the spec, but is wrong
Projects
None yet
Development

No branches or pull requests

1 participant