From fa38ae46f46f6f02ad7e1fed129fe9ef018ee944 Mon Sep 17 00:00:00 2001 From: Arthur Schiwon Date: Thu, 17 Apr 2025 17:00:31 +0200 Subject: fix(LDAP): inlcude ldapExpertUsernameAttr in general attribute list fixes corner cases in which an LDAP record might be loaded and used, where the user is still not mapped - and then this information is missing though expected. Signed-off-by: Arthur Schiwon --- apps/user_ldap/lib/User/Manager.php | 1 + 1 file changed, 1 insertion(+) diff --git a/apps/user_ldap/lib/User/Manager.php b/apps/user_ldap/lib/User/Manager.php index 125970de1e7..88a001dd965 100644 --- a/apps/user_ldap/lib/User/Manager.php +++ b/apps/user_ldap/lib/User/Manager.php @@ -109,6 +109,7 @@ class Manager { $baseAttributes = array_merge(Access::UUID_ATTRIBUTES, ['dn', 'uid', 'samaccountname', 'memberof']); $attributes = [ $this->access->getConnection()->ldapExpertUUIDUserAttr, + $this->access->getConnection()->ldapExpertUsernameAttr, $this->access->getConnection()->ldapQuotaAttribute, $this->access->getConnection()->ldapEmailAttribute, $this->access->getConnection()->ldapUserDisplayName, -- cgit v1.2.3