[pve-devel] applied: [PATCH manager] let all users retrieve the directories and tos for acme

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Jun 4 17:53:24 CEST 2018


On 6/4/18 5:12 PM, Dominik Csapak wrote:
> else all non-root users get an empty dropdown box for the directories
> and get no feedback why that is
> 
> with this, they can select it, but ultimately get an api error if the
> permissions are not sufficient
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  PVE/API2/ACMEAccount.pm | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/PVE/API2/ACMEAccount.pm b/PVE/API2/ACMEAccount.pm
> index 31c157d9..9202765a 100644
> --- a/PVE/API2/ACMEAccount.pm
> +++ b/PVE/API2/ACMEAccount.pm
> @@ -309,6 +309,7 @@ __PACKAGE__->register_method ({
>      path => 'tos',
>      method => 'GET',
>      description => "Retrieve ACME TermsOfService URL from CA.",
> +    permissions => { user => 'all' },
>      parameters => {
>  	additionalProperties => 0,
>  	properties => {
> @@ -338,6 +339,7 @@ __PACKAGE__->register_method ({
>      path => 'directories',
>      method => 'GET',
>      description => "Get named known ACME directory endpoints.",
> +    permissions => { user => 'all' },
>      parameters => {
>  	additionalProperties => 0,
>  	properties => {},
> 

applied




More information about the pve-devel mailing list