status-im/codimd
2
0
Fork 0
mirror of https://github.com/status-im/codimd.git synced 2025-01-11 16:34:33 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #562 from SISheogorath/fix/LDAP

Browse Source 
Fix LDAP problem about missing uidNumber
This commit is contained in:
Sheogorath 2017-10-27 12:48:45 +02:00 committed by GitHub
commit 881e800fd8
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
}
}, 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,