jaredhanson / passport-facebook

Facebook authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-facebook/?utm_source=github&utm_medium=referral&utm_campaign=passport-facebook&utm_content=about
MIT License
1.29k stars 447 forks source link

Authentication middleware triggers other middlewares down the line. #235

Open starleaf1 opened 6 years ago

starleaf1 commented 6 years ago

When using it like this

app.get('/auth/facebook/callback', passport.authenticate('facebook', {failureRedirect: '/fail'}), function(req, res) {
    if (req.session.redirect) {
        res.redirect(req.session.redirect);
        delete req.session.redirect;
    } else {
        res.redirect('/');
    }
});

It also triggers other middlewares down the app such as my 404 handler:

app.use(function(req, res) {
    res.status(404);
    var pageParameters = {};
    modules.logger.warn("Not found: ", req.url);
    res.render('404', pageParameters);
}