No need to log profile on every login

[perso/Immae/Projets/Nodejs/Surfer.git] / src / auth.js

diff --git a/src/auth.js b/src/auth.js
index 83d0426d4e3388604c4cc0f343fde559cb105349..39c01a11d0711ada50269afbdd7ee5b09bb6d28b 100644 (file)
--- a/src/auth.js
+++ b/src/auth.js
@@ -5,12 +5,12 @@ var passport = require('passport'),
 
 passport.serializeUser(function (user, done) {
     console.log('serializeUser', user);
-    done(null, user.id);
+    done(null, user.uid);
 });
 
 passport.deserializeUser(function (id, done) {
     console.log('deserializeUser', id);
-    done(null, { id: id });
+    done(null, { uid: id });
 });
 
 var LDAP_URL = process.env.LDAP_URL;
@@ -24,8 +24,8 @@ if (LDAP_URL && LDAP_USERS_BASE_DN) {
     exports.ldap = function (req, res, next) {
         console.log('Disable ldap auth, use developer credentials!');
 
-        if (req.query.username !== 'username') return res.send(401);
-        if (req.query.password !== 'password') return res.send(401);
+        if (req.query.username !== 'test') return res.send(401);
+        if (req.query.password !== 'test') return res.send(401);
 
         next();
     };
@@ -37,7 +37,7 @@ var opts = {
     },
     base: LDAP_USERS_BASE_DN,
     search: {
-        filter: '(uid={{username}})',
+        filter: '(|(username={{username}})(mail={{username}}))',
         attributes: ['displayname', 'username', 'mail', 'uid'],
         scope: 'sub'
     },
@@ -47,6 +47,5 @@ var opts = {
 };
 
 passport.use(new LdapStrategy(opts, function (profile, done) {
-    console.log('ldap', profile);
     done(null, profile);
 }));