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 3d2acce1eeb116a7bf3688bf3852e102beaba312..39c01a11d0711ada50269afbdd7ee5b09bb6d28b 100644 (file)
--- a/src/auth.js
+++ b/src/auth.js
@@ -3,20 +3,30 @@
 var passport = require('passport'),
     LdapStrategy = require('passport-ldapjs').Strategy;
 
+passport.serializeUser(function (user, done) {
+    console.log('serializeUser', user);
+    done(null, user.uid);
+});
+
+passport.deserializeUser(function (id, done) {
+    console.log('deserializeUser', id);
+    done(null, { uid: id });
+});
+
 var LDAP_URL = process.env.LDAP_URL;
 var LDAP_USERS_BASE_DN = process.env.LDAP_USERS_BASE_DN;
 
 if (LDAP_URL && LDAP_USERS_BASE_DN) {
     console.log('Enable ldap auth');
 
-    exports.ldap = passport.authenticate('ldap', {
-        successReturnToOrRedirect: '/',
-        failureRedirect: '/login',
-        failureFlash: true
-    });
+    exports.ldap = passport.authenticate('ldap');
 } else {
     exports.ldap = function (req, res, next) {
-        console.log('ldap auth disabled');
+        console.log('Disable ldap auth, use developer credentials!');
+
+        if (req.query.username !== 'test') return res.send(401);
+        if (req.query.password !== 'test') return res.send(401);
+
         next();
     };
 }
@@ -27,16 +37,15 @@ var opts = {
     },
     base: LDAP_USERS_BASE_DN,
     search: {
-        filter: '(uid={{username}})',
+        filter: '(|(username={{username}})(mail={{username}}))',
         attributes: ['displayname', 'username', 'mail', 'uid'],
         scope: 'sub'
     },
-    uidTag: 'uid',
+    uidTag: 'cn',
     usernameField: 'username',
     passwordField: 'password',
 };
 
 passport.use(new LdapStrategy(opts, function (profile, done) {
-    console.log('ldap', profile);
     done(null, profile);
 }));