]> git.proxmox.com Git - pve-manager.git/commitdiff
api: mark batch-execute api calls root-only
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 9 Jun 2023 08:47:43 +0000 (10:47 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Fri, 9 Jun 2023 08:47:44 +0000 (10:47 +0200)
This is weird and buggy and breaches the unpriv./priv. separation of
our api daemons, so root-only for now and possibly removal soon.

note that this had several limitations already anyway, like running
in sync context and thus failing after 30s.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
PVE/API2/Nodes.pm

index e642ab4f4031d065ae43b49c1726f7a8dbac6bb6..9269694d6a28627e3396300c0aaeef87ce284a0c 100644 (file)
@@ -464,10 +464,7 @@ __PACKAGE__->register_method({
     name => 'execute',
     path => 'execute',
     method => 'POST',
-    permissions => {
-       check => ['perm', '/nodes/{node}', [ 'Sys.Audit' ]],
-    },
-    description => "Execute multiple commands in order.",
+    description => "Execute multiple commands in order, root only.",
     proxyto => 'node',
     protected => 1, # avoid problems with proxy code
     parameters => {