proxmox-mirrors/pve-access-control
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-access-control synced 2025-08-15 13:48:20 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

api: roles: whitespace and indendation clean-ups

Browse Source 
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2023-06-08 09:22:00 +02:00
parent 8e8023b1e9
commit 43aa2489fc
1 changed files with 22 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
src/PVE/API2/Role.pm
View File

@ -85,22 +85,19 @@ __PACKAGE__->register_method ({
code => sub { code => sub {
my ($param) = @_; my ($param) = @_;
PVE::AccessControl::lock_user_config( PVE::AccessControl::lock_user_config(sub {
sub { my $usercfg = cfs_read_file("user.cfg");
my $usercfg = cfs_read_file("user.cfg"); my $role = $param->{roleid};
my $role = $param->{roleid}; die "role '$role' already exists\n" if $usercfg->{roles}->{$role};
die "role '$role' already exists\n" $usercfg->{roles}->{$role} = {};
if $usercfg->{roles}->{$role};
$usercfg->{roles}->{$role} = {}; PVE::AccessControl::add_role_privs($role, $usercfg, $param->{privs});
PVE::AccessControl::add_role_privs($role, $usercfg, $param->{privs}); cfs_write_file("user.cfg", $usercfg);
}, "create role failed");
cfs_write_file("user.cfg", $usercfg);
}, "create role failed");
return undef; return undef;
}}); }});
@ -131,20 +128,17 @@ __PACKAGE__->register_method ({
die "auto-generated role '$role' cannot be modified\n" die "auto-generated role '$role' cannot be modified\n"
if PVE::AccessControl::role_is_special($role); if PVE::AccessControl::role_is_special($role);
PVE::AccessControl::lock_user_config( PVE::AccessControl::lock_user_config(sub {
sub { my $usercfg = cfs_read_file("user.cfg");
my $usercfg = cfs_read_file("user.cfg"); die "role '$role' does not exist\n" if !$usercfg->{roles}->{$role};
die "role '$role' does not exist\n" $usercfg->{roles}->{$role} = {} if !$param->{append};
if !$usercfg->{roles}->{$role};
$usercfg->{roles}->{$role} = {} if !$param->{append}; PVE::AccessControl::add_role_privs($role, $usercfg, $param->{privs});
PVE::AccessControl::add_role_privs($role, $usercfg, $param->{privs}); cfs_write_file("user.cfg", $usercfg);
}, "update role failed");
cfs_write_file("user.cfg", $usercfg);
}, "update role failed");
return undef; return undef;
}}); }});
@ -207,19 +201,17 @@ __PACKAGE__->register_method ({
die "auto-generated role '$role' cannot be deleted\n" die "auto-generated role '$role' cannot be deleted\n"
if PVE::AccessControl::role_is_special($role); if PVE::AccessControl::role_is_special($role);
PVE::AccessControl::lock_user_config( PVE::AccessControl::lock_user_config(sub {
sub { my $usercfg = cfs_read_file("user.cfg");
my $usercfg = cfs_read_file("user.cfg");
die "role '$role' does not exist\n" die "role '$role' does not exist\n" if !$usercfg->{roles}->{$role};
if !$usercfg->{roles}->{$role};
delete ($usercfg->{roles}->{$role}); delete ($usercfg->{roles}->{$role});
# fixme: delete role from acl? # fixme: delete role from acl?
cfs_write_file("user.cfg", $usercfg); cfs_write_file("user.cfg", $usercfg);
}, "delete role failed"); }, "delete role failed");
return undef; return undef;
} }