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

Power levels always have a default of 50 for state_default #1656

Merged

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Sep 4, 2018

Rendered: see 'docs' status check


As per the proposal #1304

Closes #1304
Closes https://github.com/matrix-org/matrix-doc/issues/1192

@turt2live turt2live merged commit 16bbf46 into matrix-org:master Sep 4, 2018
@turt2live turt2live deleted the travis/s2s/simplified-pl-rules branch September 4, 2018 18:22
richvdh added a commit to matrix-org/matrix-spec that referenced this pull request Apr 5, 2023
There was substantial confusion around this, but I've done some archaeology.

Basically, this was changed back in r0.5.0 by MSC1304 and matrix-org/synapse#3397. Before that, it was indeed the case that state_default was 0 if there was no m.room.power_levels event, but that was confusing and a source of security holes, so we changed it.

matrix-org/matrix-spec-proposals#1656 changed the spec, but apparently overlooked the text in the description.

Reverts: #1478.
Fixes: #861.
clokep pushed a commit to clokep/matrix-spec that referenced this pull request May 3, 2023
…org#1479)

There was substantial confusion around this, but I've done some archaeology.

Basically, this was changed back in r0.5.0 by MSC1304 and matrix-org/synapse#3397. Before that, it was indeed the case that state_default was 0 if there was no m.room.power_levels event, but that was confusing and a source of security holes, so we changed it.

matrix-org/matrix-spec-proposals#1656 changed the spec, but apparently overlooked the text in the description.

Reverts: matrix-org#1478.
Fixes: matrix-org#861.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants