[pbs-devel] [PATCH proxmox-backup] ldap: surround user filter expression in parenthesis if not already

Lukas Wagner l.wagner at proxmox.com
Thu May 25 09:18:19 CEST 2023



On 5/24/23 10:41, Lukas Wagner wrote:
> In PVE, the `filter` attribute is surrounded in () if it is not already,
> allowing "uid=test" as well as "(uid=test)" [1].
> 
> A forum user [2] just ran into this incosistency, so I decieded to adjust
> the behavior.
> 
> [1] https://git.proxmox.com/?p=pve-common.git;a=blob;f=src/PVE/LDAP.pm;h=ff98e367e63265bf76c0f302847c3749eea095a6;hb=HEAD#l115
> [2] https://forum.proxmox.com/threads/ldap-query-for-security-group-members.127882/
> 
> Signed-off-by: Lukas Wagner <l.wagner at proxmox.com>
> ---
>   proxmox-ldap/src/lib.rs | 10 +++++++++-

      ^
Sorry, wrong repo in subject, should of course be `proxmox`! :)

-- 
- Lukas





More information about the pbs-devel mailing list