[pve-devel] [PATCH] fix #6932: api: acme: add digest and plugin to return schema

[Nicolas Frey]

Renamed $acme_challenge_api_create_and_return_schema in v2

Superseded-by: https://lore.proxmox.com/pve-devel/20251015111740.122773-1-n.frey@proxmox.com/T/#u

On 10/15/25 12:04 PM, Nicolas Frey wrote:
> fixes regression from `427e2627 api: add ACME plugin return schema`
> by adding digest and plugin parameters and deleting id parameter, as
> it is replaced by plugin on return schemas. This mismatch would
> result in `400 Result verification failed` on index and
> get_plugin_config.
> 
> Fixes: https://bugzilla.proxmox.com/show_bug.cgi?id=6932
> Signed-off-by: Nicolas Frey <n.frey at proxmox.com>
> ---
>  PVE/API2/ACMEPlugin.pm | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/PVE/API2/ACMEPlugin.pm b/PVE/API2/ACMEPlugin.pm
> index c12b75c5..1754d9f0 100644
> --- a/PVE/API2/ACMEPlugin.pm
> +++ b/PVE/API2/ACMEPlugin.pm
> @@ -50,7 +50,17 @@ my $modify_cfg_for_api = sub {
>      return $plugin_cfg;
>  };
>  
> -my $acme_challenge_api_create_and_return_schema = PVE::ACME::Challenge->createSchema();
> +my $acme_challenge_create_schema = PVE::ACME::Challenge->createSchema();
> +my $acme_challenge_api_create_and_return_schema = {
> +    type => "object",
> +    properties => {
> +        PVE::ACME::Challenge->createSchema()->{properties}->%*,
> +        digest => get_standard_option('pve-config-digest'),
> +        plugin => get_standard_option('pve-acme-pluginid'),
> +    },
> +};
> +# replaced by plugin property
> +delete $acme_challenge_api_create_and_return_schema->{properties}->{id};
>  
>  __PACKAGE__->register_method({
>      name => 'index',
> @@ -126,7 +136,7 @@ __PACKAGE__->register_method({
>          check => ['perm', '/', ['Sys.Modify']],
>      },
>      protected => 1,
> -    parameters => $acme_challenge_api_create_and_return_schema,
> +    parameters => $acme_challenge_create_schema,
>      returns => {
>          type => "null",
>      },