Flood-UI / flood

A web UI for rTorrent, qBittorrent and Transmission with a Node.js backend and React frontend. Migrate to v4: https://github.com/jesec/flood/wiki/Migrate-from-older-versions-of-Flood.
https://flood.js.org
GNU General Public License v3.0
1.81k stars 173 forks source link

User Management #717

Closed zawapete closed 5 years ago

zawapete commented 6 years ago

Note

Summary

At the moment the Authentication tab is open to every user, regardless the use is admin or not (isAdmin flag introduced with multi rtorrent instance feature)

Expected Behavior

Authentication tab should not render user management for not admin users Server CRUD on user should be forbidden for not admin users

Current Behavior

Every user is able to manage other users regardless of the admin status

Context

Shared seedbox administrator cannot rely totally on flood since it gives more permissions that it should to not administrator users

noraj commented 6 years ago

Related PR : #718

bqv commented 5 years ago

Can this be fixed for the "flood can't connect" page too

bqv commented 5 years ago

The page to configure flood host and port is shown even to non admins if it fails even once, and I can't get it to stop without deleting and recreating the user

zawapete commented 5 years ago

sorry, closed by mistake