]> git.proxmox.com Git - pve-manager.git/commit - PVE/API2/Backup.pm
api: backup: require Datastore.Allocate on storage
authorFiona Ebner <f.ebner@proxmox.com>
Wed, 16 Nov 2022 14:04:34 +0000 (15:04 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Wed, 7 Jun 2023 14:47:08 +0000 (16:47 +0200)
commitb6e561304a6145c7637f354fb4355198bb11c5ba
tree50ae8ba103b4b1934e6d5aec30cbea7af704f5cb
parent9f65a584b7781b7bb7ed6571513110f5ab0a18e8
api: backup: require Datastore.Allocate on storage

In particular this ensures that the user is allowed to remove data on
the storage, because configuring low retention results in removed
older backups. Of course setting the storage itself also needs to
require the same privilege then.

This is a breaking API change, but it seems sensible to require
permissions on the affected storage too.

Jobs with a dumpdir setting can be configured by root only.

Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
PVE/API2/Backup.pm