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

room upgrade doesn't transfer non-standard power structure #5580

Closed
grinapo opened this issue Jun 28, 2019 · 6 comments
Closed

room upgrade doesn't transfer non-standard power structure #5580

grinapo opened this issue Jun 28, 2019 · 6 comments
Assignees
Labels
A-Room-Upgrades z-feature (Deprecated Label) z-p2 (Deprecated Label)

Comments

@grinapo
Copy link

grinapo commented Jun 28, 2019

Description

A room with nonstandard power structure cannot be upgraded, since the power levels are not moved over. In fact, upgrade results:

{
    "errcode": "M_FORBIDDEN",
    "error": "You don't have permission to add ops level greater than your own"
}

Steps to reproduce

  • crate a room with various levels above 100, with owners and members above, too.
  • room/.../upgrade (I have used from v5 to v5)
  • kaboom

Version information

  • Homeserver: grin.hu
  • Version: 1.0.0
  • Install method: self-hacked debian package
  • Platform: debian/buster
@richvdh
Copy link
Member

richvdh commented Jun 30, 2019

crate a room with various levels above 100, with owners and members above, too.

How do you do this?

@babolivier
Copy link
Contributor

babolivier commented Jul 1, 2019

How do you do this?

Via the initial_state param in the /createRoom request (I've just successfully granted myself PL 500 in a new room that way).

@grinapo
Copy link
Author

grinapo commented Jul 1, 2019

Related: https://github.com/matrix-org/matrix-doc/issues/2149

@babolivier I have used power_level_content_override but it's the same.

@babolivier
Copy link
Contributor

babolivier commented Jul 1, 2019

Ah, yeah, I forgot that was a thing. Anyway, yes, the max PL is overridable at room creation time.

@neilisfragile neilisfragile added z-bug (Deprecated Label) security labels Jul 3, 2019
@richvdh
Copy link
Member

richvdh commented Jul 3, 2019

I'm not convinced this is worthy of a 'security' label

@richvdh richvdh added z-feature (Deprecated Label) A-Room-Upgrades and removed z-bug (Deprecated Label) security labels Jul 3, 2019
@neilisfragile neilisfragile added the z-p2 (Deprecated Label) label Jul 31, 2019
@anoadragon453
Copy link
Member

Should be fixed by #6237

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Room-Upgrades z-feature (Deprecated Label) z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

5 participants