From d7858f44dcc79aa29368c447777b9588e7ebdbc5 Mon Sep 17 00:00:00 2001 From: Ben Ramey Date: Tue, 10 Jun 2014 21:21:24 -0500 Subject: [PATCH] Add delete response for logout --- api/controllers/SessionController.js | 10 +++++----- config/policies.js | 4 ++++ config/routes.js | 1 + 3 files changed, 10 insertions(+), 5 deletions(-) diff --git a/api/controllers/SessionController.js b/api/controllers/SessionController.js index 87e83df..2aca5a1 100644 --- a/api/controllers/SessionController.js +++ b/api/controllers/SessionController.js @@ -19,23 +19,23 @@ var passport = require('passport'); module.exports = { create: function(req,res) { + console.log('create'); passport.authenticate('local', function(err, user, info) { if(err || !user) { - res.redirect('/login'); - return; + return res.json(false); } req.logIn(user, function(err) { if (err) { - res.redirect('/login'); + return res.json(false); } - return res.redirect('/'); + return res.json(true); }); })(req, res); }, destroy: function(req, res) { req.logout(); - res.send('logout successful'); + res.send(true); }, /** diff --git a/config/policies.js b/config/policies.js index cfa201d..d8be365 100644 --- a/config/policies.js +++ b/config/policies.js @@ -36,6 +36,10 @@ module.exports.policies = { SessionController: { 'create': true, 'destroy': 'isAuthenticated' + }, + + UserController: { + '*': true } }; diff --git a/config/routes.js b/config/routes.js index 3b37ff9..eeaec09 100644 --- a/config/routes.js +++ b/config/routes.js @@ -42,6 +42,7 @@ module.exports.routes = { // session 'post /login': 'SessionController.create', 'post /logout': 'SessionController.destroy', + 'delete /logout': 'SessionController.destroy', /* // But what if you want your home page to display