[pve-devel] applied: [PATCH pve-manager] api: notification targets: fix permission check for POST/PUT

[Thomas Lamprecht]

Am 29.11.24 um 09:59 schrieb Lukas Wagner:
> This fixes the error:
>   unknown permission test at /usr/share/perl5/PVE/RPCEnvironment.pm line 536. (500)
> which occured when trying to create or update a notification target.
> 
> The cause was a permission 'check' parameter for the API handlers which was nested
> one level too deep by accident.
> 
> This regression was introduced in a previous commit which raised the
> needed permissions for notification target management. It likely went
> unnoticed because the permission check is skipped for root at pam, so
> the error occurs only if using another user.
> 
> Reported in the community forum:
> https://forum.proxmox.com/threads/158101
> 
> Fixes: a3fe9c54 ("api: notifications: require powerful privileges for target management")
> Signed-off-by: Lukas Wagner <l.wagner at proxmox.com>
> ---
>  PVE/API2/Cluster/Notifications.pm | 96 +++++++++++++------------------
>  1 file changed, 40 insertions(+), 56 deletions(-)
> 
>

applied, thanks!