Skip to content

Commit 1eb9000

Browse files
committed
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 <blizzz@arthur-schiwon.de>
1 parent 47f1950 commit 1eb9000

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

apps/user_ldap/lib/User/Manager.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ public function getAttributes($minimal = false) {
109109
$baseAttributes = array_merge(Access::UUID_ATTRIBUTES, ['dn', 'uid', 'samaccountname', 'memberof']);
110110
$attributes = [
111111
$this->access->getConnection()->ldapExpertUUIDUserAttr,
112+
$this->access->getConnection()->ldapExpertUsernameAttr,
112113
$this->access->getConnection()->ldapQuotaAttribute,
113114
$this->access->getConnection()->ldapEmailAttribute,
114115
$this->access->getConnection()->ldapUserDisplayName,

0 commit comments

Comments
 (0)