成功,但仍然使用failureRedirect - passport

时间:2015-04-08 09:29:14

标签: node.js express passport.js passport-local

app.post('/signup', passport.authenticate('local-signup', {
  successRedirect: '/',
  failureRedirect: '/signup',
  failureFlash: true
}));

这是路线。

passport.use('local-signup', new LocalStrategy({
    usernameField: 'email',
    passwordField: 'password',
    passReqToCallback: true
},
function(req, email, password, done) {
    process.nextTick(function() {
        User.findOne({ 'local.email': email }, function(err, user) {
            if(err)
                return done(err);

            if(user) {
                return done(null, false, req.flash('signupMessage', 'That email is already taken'));
            } else {
                var newUser = new User();

                newUser.local.email = email;
                newUser.local.password = newUser.generateHash(password);

                newUser.save(function(err) {
                    done(err, user);
                });
            }
        });
    });
}));

这是护照配置文件。表单按预期工作并使用我的mongo架构保存,但它使用的是failureRedirect而不是successRedirect,有人注意到了什么问题吗?

1 个答案:

答案 0 :(得分:1)

当你在新创建的用户上调用save函数时,你忘记将第二个参数传递给它的回调,这是从db返回的用户文档。

从你当前的代码逻辑来看,你上次完成的函数中的用户是指来自findOne的回调,因此永远不会到达那里,因为如果那个用户在那时从服务器返回它将执行{ {1}}。

确保传递第二个参数以便像这样保存:

if (user) { ... }