jaredhanson / passport-linkedin

LinkedIn authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-linkedin/?utm_source=github&utm_medium=referral&utm_campaign=passport-linkedin&utm_content=about
MIT License
141 stars 71 forks source link

Session problem #24

Closed igordeoliveirasa closed 7 years ago

igordeoliveirasa commented 7 years ago

Hi, I really would like not to use session. However, I've been receiving this message:

OAuth authentication requires session support. Did you forget to use express-session middleware?

My code follows:

passport.use(new` LocalStrategy(
    function(username, password, done) {
        var password = password ? '' + password : null;
        return User
            .findOne({username:username})
            .populate('roles')
            .then(function(user){
                if (!user) {
                    return done(null, false);
                }
                else if (user) {
                    if (!user.password && user.facebookId) {
                        return done(null, false);
                    }
                    else if (!bcrypt.compareSync(password, user.password)) {
                        return done(null, false);
                    }
                    else {
                        return done(null, user);
                    }
                }
            },function(err){
                return done(err);
            });
    }
));

and

passport.authenticate('linkedin', {session: false}, function(err, user, info) {// code continues...

igordeoliveirasa commented 7 years ago

Don't worry! My fault!

samcus commented 7 years ago

@igordeoliveirasa I added '{session: false}' and I still get the error about session being required, any ideas?

samcus commented 7 years ago

@igordeoliveirasa I ended up using https://github.com/auth0/passport-linkedin-oauth2 instead.