From f93a14e3e1cb8f6589feb101d6b3b0effaf8ea61 Mon Sep 17 00:00:00 2001 From: Sheogorath Date: Sat, 7 Oct 2017 01:49:43 +0200 Subject: [PATCH] Fix LDAP problem about missing uidNumber Signed-off-by: Sheogorath --- lib/web/auth/ldap/index.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/web/auth/ldap/index.js b/lib/web/auth/ldap/index.js index 766c5cbcc..9a63578aa 100644 --- a/lib/web/auth/ldap/index.js +++ b/lib/web/auth/ldap/index.js @@ -23,9 +23,10 @@ passport.use(new LDAPStrategy({ tlsOptions: config.ldap.tlsOptions || null } }, function (user, done) { + var uuid = user.uidNumber || user.uid || user.sAMAccountName var profile = { - id: 'LDAP-' + user.uidNumber, - username: user.uid, + id: 'LDAP-' + uuid, + username: uuid, displayName: user.displayName, emails: user.mail ? [user.mail] : [], avatarUrl: null,