From 04e774a748f00993d254671e068fcb58ba567da1 Mon Sep 17 00:00:00 2001 From: nanalelfe Date: Fri, 22 Jul 2016 20:20:04 -0400 Subject: Leftover commits --- routes/user.js | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'routes') diff --git a/routes/user.js b/routes/user.js index a8e90bb..ccc4f87 100644 --- a/routes/user.js +++ b/routes/user.js @@ -7,7 +7,7 @@ var csrfProtection = csrf(); router.use(csrfProtection); // router is protected -router.get('/logout', function (req, res, next) { +router.get('/logout', loggedIn, function (req, res, next) { req.logout(); res.redirect('/'); }); @@ -17,12 +17,12 @@ router.get('/user_profile', loggedIn, function(req, res, next) { res.render('user_profile_alt'); }); -router.get('/signup', function(req, res, next) { +router.get('/signup', loggedOut, function(req, res, next) { res.render('signup', {csrfToken: req.csrfToken(), success: req.session.success, errors: req.session.errors}); req.session.errors = null; }); -router.get('/signup/failed', function(req, res, next) { +router.get('/signup/failed', loggedOut, function(req, res, next) { var msg = req.flash('error'); res.render('signup', {csrfToken: req.csrfToken(), @@ -31,8 +31,7 @@ router.get('/signup/failed', function(req, res, next) { flashMsg: msg}); }); - -router.get('/signin', function (req, res, next) { +router.get('/signin', loggedOut, function (req, res, next) { var msg = req.flash('error'); res.render('signin', { csrfToken: req.csrfToken(), @@ -42,7 +41,6 @@ router.get('/signin', function (req, res, next) { }); }); - router.post('/signup', loggedOut, function(req, res, next) { req.check('fname', 'Please enter a valid first name.').notEmpty().withMessage('First name required.').isAlpha(); req.check('lname', 'Please enter a valid first name.').notEmpty().withMessage('Last name required.').isAlpha(); @@ -97,8 +95,6 @@ router.post('/signin', loggedOut, function(req, res, next) { module.exports = router; - - function loggedIn(req, res, next) { if (req.isAuthenticated()) { return next(); @@ -106,7 +102,6 @@ function loggedIn(req, res, next) { res.redirect('/'); } - function loggedOut(req, res, next) { if (!req.isAuthenticated()) { return next(); -- cgit v1.2.3 From 3438e19cea85ae530afd1535fe0384a8beebec90 Mon Sep 17 00:00:00 2001 From: nanalelfe Date: Fri, 22 Jul 2016 20:30:27 -0400 Subject: Fixed minor issue --- routes/user.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'routes') diff --git a/routes/user.js b/routes/user.js index 698c4f8..39b41d2 100644 --- a/routes/user.js +++ b/routes/user.js @@ -56,7 +56,7 @@ router.post('/signup', loggedOut, function(req, res, next) { if (errors) { req.session.errors = errors; req.session.success = false; - res.redirect('/signup'); + res.redirect('/user/signup'); } else { console.log("GOT SUCCESS"); passport.authenticate('local_signup', { -- cgit v1.2.3 From 5240b6ff1bd881b482caba3dc3c1b30597fb1353 Mon Sep 17 00:00:00 2001 From: nanalelfe Date: Fri, 22 Jul 2016 22:12:02 -0400 Subject: Fixed session storage. Fixed optional phone number in signup --- routes/user.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'routes') diff --git a/routes/user.js b/routes/user.js index 39b41d2..840930a 100644 --- a/routes/user.js +++ b/routes/user.js @@ -49,7 +49,9 @@ router.post('/signup', loggedOut, function(req, res, next) { req.check('password', "Password should be between 6 and 12 characters.") .notEmpty().withMessage('Password required').isLength({min: 6, max: 12}); req.check('password', "The confirmation password doesn't match.").equals(req.body.confirmPassword); - req.check('phone_num', 'Please enter a valid phone number').isMobilePhone('en-CA').optional(); + if (req.body.phone_num){ + req.check('phone_num', 'Please enter a valid phone number').isMobilePhone('en-CA'); + } // password has to be at least 4 characters long var errors = req.validationErrors(); -- cgit v1.2.3