Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Ensure isLoggedIn and isLoggedOut middlewares #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Ensure isLoggedIn and isLoggedOut middlewares #19

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e345655
Ensure isLoggedIn and isLoggedOut middlewares
skotchio Feb 7, 2015
d446804
Merge pull request #1 from skotchio/skotchio-patch-1
skotchio Feb 7, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 28 additions & 13 deletions app/routes.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ module.exports = function(app, passport) {
});

// LOGOUT ==============================
app.get('/logout', function(req, res) {
app.get('/logout', isLoggedIn, function(req, res) {
req.logout();
res.redirect('/');
});
Expand All @@ -27,25 +27,25 @@ module.exports = function(app, passport) {
// locally --------------------------------
// LOGIN ===============================
// show the login form
app.get('/login', function(req, res) {
app.get('/login', isLoggedOut, function(req, res) {
res.render('login.ejs', { message: req.flash('loginMessage') });
});

// process the login form
app.post('/login', passport.authenticate('local-login', {
app.post('/login', isLoggedOut, passport.authenticate('local-login', {
successRedirect : '/profile', // redirect to the secure profile section
failureRedirect : '/login', // redirect back to the signup page if there is an error
failureFlash : true // allow flash messages
}));

// SIGNUP =================================
// show the signup form
app.get('/signup', function(req, res) {
app.get('/signup', isLoggedOut, function(req, res) {
res.render('signup.ejs', { message: req.flash('signupMessage') });
});

// process the signup form
app.post('/signup', passport.authenticate('local-signup', {
app.post('/signup', isLoggedOut, passport.authenticate('local-signup', {
successRedirect : '/profile', // redirect to the secure profile section
failureRedirect : '/signup', // redirect back to the signup page if there is an error
failureFlash : true // allow flash messages
Expand All @@ -54,10 +54,11 @@ module.exports = function(app, passport) {
// facebook -------------------------------

// send to facebook to do the authentication
app.get('/auth/facebook', passport.authenticate('facebook', { scope : 'email' }));
app.get('/auth/facebook', isLoggedOut, passport.authenticate('facebook', { scope : 'email' }));

// handle the callback after facebook has authenticated the user
app.get('/auth/facebook/callback',
isLoggedOut,
passport.authenticate('facebook', {
successRedirect : '/profile',
failureRedirect : '/'
Expand All @@ -66,10 +67,11 @@ module.exports = function(app, passport) {
// twitter --------------------------------

// send to twitter to do the authentication
app.get('/auth/twitter', passport.authenticate('twitter', { scope : 'email' }));
app.get('/auth/twitter', isLoggedOut, passport.authenticate('twitter', { scope : 'email' }));

// handle the callback after twitter has authenticated the user
app.get('/auth/twitter/callback',
isLoggedOut,
passport.authenticate('twitter', {
successRedirect : '/profile',
failureRedirect : '/'
Expand All @@ -79,10 +81,11 @@ module.exports = function(app, passport) {
// google ---------------------------------

// send to google to do the authentication
app.get('/auth/google', passport.authenticate('google', { scope : ['profile', 'email'] }));
app.get('/auth/google', isLoggedOut, passport.authenticate('google', { scope : ['profile', 'email'] }));

// the callback after google has authenticated the user
app.get('/auth/google/callback',
isLoggedOut,
passport.authenticate('google', {
successRedirect : '/profile',
failureRedirect : '/'
Expand All @@ -93,10 +96,10 @@ module.exports = function(app, passport) {
// =============================================================================

// locally --------------------------------
app.get('/connect/local', function(req, res) {
app.get('/connect/local', isLoggedIn, function(req, res) {
res.render('connect-local.ejs', { message: req.flash('loginMessage') });
});
app.post('/connect/local', passport.authenticate('local-signup', {
app.post('/connect/local', isLoggedIn, passport.authenticate('local-signup', {
successRedirect : '/profile', // redirect to the secure profile section
failureRedirect : '/connect/local', // redirect back to the signup page if there is an error
failureFlash : true // allow flash messages
Expand All @@ -105,10 +108,11 @@ module.exports = function(app, passport) {
// facebook -------------------------------

// send to facebook to do the authentication
app.get('/connect/facebook', passport.authorize('facebook', { scope : 'email' }));
app.get('/connect/facebook', isLoggedIn, passport.authorize('facebook', { scope : 'email' }));

// handle the callback after facebook has authorized the user
app.get('/connect/facebook/callback',
isLoggedIn,
passport.authorize('facebook', {
successRedirect : '/profile',
failureRedirect : '/'
Expand All @@ -117,10 +121,11 @@ module.exports = function(app, passport) {
// twitter --------------------------------

// send to twitter to do the authentication
app.get('/connect/twitter', passport.authorize('twitter', { scope : 'email' }));
app.get('/connect/twitter', isLoggedIn, passport.authorize('twitter', { scope : 'email' }));

// handle the callback after twitter has authorized the user
app.get('/connect/twitter/callback',
isLoggedIn,
passport.authorize('twitter', {
successRedirect : '/profile',
failureRedirect : '/'
Expand All @@ -130,10 +135,11 @@ module.exports = function(app, passport) {
// google ---------------------------------

// send to google to do the authentication
app.get('/connect/google', passport.authorize('google', { scope : ['profile', 'email'] }));
app.get('/connect/google', isLoggedIn, passport.authorize('google', { scope : ['profile', 'email'] }));

// the callback after google has authorized the user
app.get('/connect/google/callback',
isLoggedIn,
passport.authorize('google', {
successRedirect : '/profile',
failureRedirect : '/'
Expand Down Expand Up @@ -193,3 +199,12 @@ function isLoggedIn(req, res, next) {

res.redirect('/');
}

// route middleware to ensure user is logged out
function isLoggedOut (req, res, next) {
if (req.isUnauthenticated()) {
return next();
}

res.redirect('/');
}