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

Loosen type of call member event state keys #1885

Merged
merged 1 commit into from
Aug 10, 2024

Conversation

AndrewFerr
Copy link
Contributor

Allow MSC3401-style call membership state events to have state keys that are arbitrary strings, and not just pure MXIDs.

This allows for state keys that are suffixed with the sender's device ID, as well as ones prefixed by an underscore to bypass auth rules over state keys that start with a @ but do not strictly equal an MXID.

Allow MSC3401-style call membership state events to have state keys that
are arbitrary strings, and not just pure MXIDs.

This allows for state keys that are suffixed with the sender's device
ID, as well as ones prefixed by an underscore to bypass auth rules over
state keys that start with a `@` but do not strictly equal an MXID.
Copy link
Contributor

@zecakeh zecakeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Eventually, we'll probably want to use an enum with the different possibilities, but this should be good for now.

@zecakeh zecakeh merged commit 82417e3 into ruma:main Aug 10, 2024
22 checks passed
@AndrewFerr AndrewFerr deleted the call-member-state-key-type branch August 12, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants