[pve-devel] applied: [PATCH access-control 1/2] ldap: Allow quoted values for DN attribute values
Thomas Lamprecht
t.lamprecht at proxmox.com
Mon Mar 20 16:09:38 CET 2023
Am 15/03/2023 um 10:54 schrieb Dominik Csapak:
> hi,
>
> so high level comment:
> i'd write most of what you wrote in the cover letter here in the commit message,
> makes it much more convenient to find it only via git ;)
>
> also i'm missing a bit the rationale for how the regex was chosen, besides
> that it works in some conditions
>
> further comment inline
>
> On 1/31/23 13:50, Christoph Heiss wrote:
>> Signed-off-by: Christoph Heiss <c.heiss at proxmox.com>
>> ---
>> src/PVE/Auth/LDAP.pm | 8 +++++---
>> 1 file changed, 5 insertions(+), 3 deletions(-)
>>
>>
applied, with Dominiks R-b/T-b and more info and references added to the commit
message, thanks!
More information about the pve-devel
mailing list