[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