depau/HackMD
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix LDAP problem about missing uidNumber

Browse source 
Signed-off-by: Sheogorath <sheogorath@shivering-isles.com>
This commit is contained in:
Sheogorath 2017-10-07 01:49:43 +02:00
parent d1d6d5810b
commit f93a14e3e1
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/web/auth/ldap/index.js
View file

@ -23,9 +23,10 @@ passport.use(new LDAPStrategy({
tlsOptions: config.ldap.tlsOptions || null tlsOptions: config.ldap.tlsOptions || null
} }
}, function (user, done) { }, function (user, done) {
var uuid = user.uidNumber || user.uid || user.sAMAccountName
var profile = { var profile = {
id: 'LDAP-' + user.uidNumber, id: 'LDAP-' + uuid,
username: user.uid, username: uuid,
displayName: user.displayName, displayName: user.displayName,
emails: user.mail ? [user.mail] : [], emails: user.mail ? [user.mail] : [],
avatarUrl: null, avatarUrl: null,