[pve-devel] [PATCH v3 access-control++ 00/17] SuperUser privilege

Oguz Bektas o.bektas at proxmox.com
Wed Apr 6 13:57:17 CEST 2022


v2->v3:
* fixed some incorrect checks in qemu-server
* further restrict changing passwords and tfa settings for superusers
* gui fix for lxc features
* slight improvements to docs, added some notes

see the separate patches for further details :)

intentionally left out ceph and cluster-related endpoints for checking SU
privileges:

* addnode
* copy
* create
* delnode
* index
* join

* register_account
* deactivate_account

* destroyosd
* createosd

i left these endpoints alone since we have plans to introduce separate
privileges for cluster-related actions


 access-control: Oguz Bektas (5):
  add "SuperAdministrator" role with the new "SuperUser" privilege
  RPC env: add SuperUser API permission for GUI capabilities
  api: acl: only allow granting SU privilege if user already has it
  api: roles: only allow modifying roles to add/remove SU if user has SU themselves
  api: allow superusers to edit tfa and password settings

 src/PVE/API2/ACL.pm           | 17 +++++++++++++++++
 src/PVE/API2/AccessControl.pm | 23 +++++++++++++++--------
 src/PVE/API2/Role.pm          | 21 +++++++++++++++++++++
 src/PVE/API2/TFA.pm           | 11 ++++++++++-
 src/PVE/AccessControl.pm      |  9 ++++++---
 src/PVE/RPCEnvironment.pm     | 29 ++++++++++++++++++++++-------
 6 files changed, 91 insertions(+), 19 deletions(-)

 qemu-server: Oguz Bektas (3):
  api: allow SU privileged users to edit root-only options for VM configs
  migration tests: mock $rpcenv->check subroutine
  api: allow superusers to use 'skiplock' option

 PVE/API2/Qemu.pm             | 133 +++++++++++++++++++++--------------
 test/MigrationTest/QmMock.pm |   5 ++
 2 files changed, 84 insertions(+), 54 deletions(-)

 manager: Oguz Bektas (6):
  api: backup: allow SUs to use 'tmpdir', 'dumpdir' and 'script' options
  api: vzdump: allow SUs to use 'bwlimit' and 'ionice' parameters
  api: always show login prompt for non-root users on terminal proxy calls
  ui: include "SuperUser" in privilege selector
  ui: lxc features: check for SU instead of 'root at pam'
  ui: adapt sensible 'root at pam' checks to SU

 PVE/API2/Backup.pm                      | 11 +++++++----
 PVE/API2/Nodes.pm                       | 11 ++++++++---
 PVE/API2/VZDump.pm                      |  8 +++++---
 www/manager6/form/PrivilegesSelector.js |  2 +-
 www/manager6/lxc/Options.js             |  8 ++++++--
 www/manager6/lxc/Resources.js           |  2 +-
 www/manager6/window/Migrate.js          |  4 ++--
 7 files changed, 30 insertions(+), 16 deletions(-)

 container: Oguz Bektas (1):
  fix #2582: api: add checks for 'SuperUser' privilege for root-only options

 src/PVE/API2/LXC.pm        | 19 +++++++++----------
 src/PVE/API2/LXC/Config.pm |  2 +-
 src/PVE/API2/LXC/Status.pm | 12 ++++++++----
 src/PVE/LXC.pm             | 21 ++++++++++++---------
 src/PVE/LXC/Create.pm      |  2 +-
 5 files changed, 31 insertions(+), 25 deletions(-)

 storage: Oguz Bektas (1):
  check_volume_access: allow superusers to pass arbitrary fs paths

 PVE/Storage.pm | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

 docs: Oguz Bektas (1):
  pveum: add SU privilege and SA role

 pveum.adoc | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

-- 
2.30.2





More information about the pve-devel mailing list