[pve-devel] applied: [PATCH access-control] api: domains: fix setting only the realm authentication password
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri Jul 21 16:14:17 CEST 2023
On 20/07/2023 16:48, Christoph Heiss wrote:
> Previously, if one tried to only update e.g. the bind password for an
> LDAP realm, it would fail with an rather unhelpful error message:
>
> # pveum realm modify ldap -password foo
> update auth server failed: no options specified
>
> The root cause was that the `password` parameter was removed early from
> the parameter object, which than would fail the check whether it is
> empty or not.
>
> Thus, additionally check if only `password` was specified and if so,
> allow it.
>
> Reported-by: Friedrich Weber <f.weber at proxmox.com>
> Signed-off-by: Christoph Heiss <c.heiss at proxmox.com>
> ---
> src/PVE/API2/Domains.pm | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>
applied, thanks!
More information about the pve-devel
mailing list