Skip to content

Loosen type of call member event state keys#1885

Merged
zecakeh merged 1 commit intoruma:mainfrom
AndrewFerr:call-member-state-key-type
Aug 10, 2024
Merged

Loosen type of call member event state keys#1885
zecakeh merged 1 commit intoruma:mainfrom
AndrewFerr:call-member-state-key-type

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
@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