From fcb6f902932e95861a96331ed651bc69c7482c66 Mon Sep 17 00:00:00 2001 From: Mikael Korpela Date: Fri, 29 May 2015 00:36:07 +0300 Subject: [PATCH] Remove un-used hasAuthorization and requiresLogin Looks like these aren't needed now that we have ACL. --- .../users.authorization.server.controller.js | 32 ------------------- 1 file changed, 32 deletions(-) diff --git a/modules/users/server/controllers/users/users.authorization.server.controller.js b/modules/users/server/controllers/users/users.authorization.server.controller.js index f0d1b9ca5e..9ad24fc3ef 100644 --- a/modules/users/server/controllers/users/users.authorization.server.controller.js +++ b/modules/users/server/controllers/users/users.authorization.server.controller.js @@ -20,35 +20,3 @@ exports.userByID = function(req, res, next, id) { next(); }); }; - -/** - * Require login routing middleware - */ -exports.requiresLogin = function(req, res, next) { - if (!req.isAuthenticated()) { - return res.status(401).send({ - message: 'User is not logged in' - }); - } - - next(); -}; - -/** - * User authorizations routing middleware - */ -exports.hasAuthorization = function(roles) { - var _this = this; - - return function(req, res, next) { - _this.requiresLogin(req, res, function() { - if (_.intersection(req.user.roles, roles).length) { - return next(); - } else { - return res.status(403).send({ - message: 'User is not authorized' - }); - } - }); - }; -};