[pve-devel] applied: [PATCH qemu-server] allow reading snapshot config for VM.Audit

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Jan 31 20:33:07 CET 2020


On 1/31/20 11:40 AM, Dominik Csapak wrote:
> VM.Audit can see the current config and the list of snapshots
> already, so there is no real reason to disallow
> the config of snapshots
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  PVE/API2/Qemu.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm
> index 89e2477..e15c0c3 100644
> --- a/PVE/API2/Qemu.pm
> +++ b/PVE/API2/Qemu.pm
> @@ -3841,7 +3841,7 @@ __PACKAGE__->register_method({
>      proxyto => 'node',
>      description => "Get snapshot configuration",
>      permissions => {
> -	check => ['perm', '/vms/{vmid}', [ 'VM.Snapshot', 'VM.Snapshot.Rollback' ], any => 1],
> +	check => ['perm', '/vms/{vmid}', [ 'VM.Snapshot', 'VM.Snapshot.Rollback', 'VM.Audit' ], any => 1],
>      },
>      parameters => {
>  	additionalProperties => 0,
> 

applied, thanks!




More information about the pve-devel mailing list