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

[Bug]: Lizmap admin rights management: security flaw? #5218

Open
1 task done
meyerlor opened this issue Jan 16, 2025 · 2 comments
Open
1 task done

[Bug]: Lizmap admin rights management: security flaw? #5218

meyerlor opened this issue Jan 16, 2025 · 2 comments

Comments

@meyerlor
Copy link

What is the bug? (in English)

Recently i started to work more with the user rights management - i discovered one thing which troubles me a bit: If i give a user (or a usergroup) the right to change the rights of other users, this users can then promote other users (and themselfes) into the "admins" group and therefore escalate their rights. Can you somehow prevent that? As a user, i should not be able to grant rights which are higher than my currently owned rights i think..

Steps to reproduce the issue

Lizmap Admin Interface
Grant User the "Change User Rights" permission
-> The granted user can promote other users without any restrictions

Versions, safeguards, check summary etc

Versions :

  • Lizmap Web Client : 3.8.4 - commit be354fc be354fcbc
  • Lizmap plugin : 4.4.6
  • QGIS Desktop : 3.34.13
  • QGIS Server : 3.34.14
  • Py-QGIS-Server : not used
  • QGIS Server plugin atlasprint : 3.4.1
  • QGIS Server plugin lizmap_server : 2.12.0
  • QGIS Server plugin wfsOutputExtension : 1.8.2
List of Lizmap Web Client modules :
* altiProfil : 0.5.8 * altiProfilAdmin : 0.5.8
List of safeguards :
* Mode : normal
* Allow parent folder : no
* Prevent other drive : yes
* Prevent PG service : yes
* Prevent PG Auth DB : yes
* Force PG user&pass : yes
* Prevent ECW : yes

Check Lizmap plugin

  • I have done the step just before in the Lizmap QGIS desktop plugin before opening this ticket. Otherwise, my ticket is not considered valid and might get closed.

Operating system

Ubuntu 22.04

Browsers

Firefox

Browsers version

Firefox 89

Relevant log output

No response

@laurentj
Copy link
Collaborator

laurentj commented Jan 24, 2025

If you give the right to a user to modify rights of other users, then you consider this user as an administrator user. Having the right to modify rights, it is a "super power". So you give him the power to modify rights of other users (or groups) and to put them into groups that have this rights.

If you don't trust this user, don't give him this super power.

As a user, i should not be able to grant rights which are higher than my currently owned rights

There is no hierarchy in groups or rights. Groups are just a list of rights.

@meyerlor
Copy link
Author

Hm okay well - I wanted the user to manage other users (allow them to edit layer XY as it is in group policy XY) - i trust him on this. But knowing he is lacking any IT knowledge, i fear he might accidently give someone (or himself) lizmap-administrative rights, which i would see as potentially dangerous.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants