Просмотр исходного кода

added null check on user when checking permissions

Henry Oswald 11 лет назад
Родитель
Сommit
f61d97a4f6
1 измененных файлов с 3 добавлено и 1 удалено
  1. 3 1
      services/web/app/coffee/managers/SecurityManager.coffee

+ 3 - 1
services/web/app/coffee/managers/SecurityManager.coffee

@@ -89,7 +89,9 @@ module.exports = SecurityManager =
 
 	requestIsOwner : (req, res, next)->
 		getRequestUserAndProject req, res, {}, (err, user, project)->
-			if userIsOwner user, project || user.isAdmin
+			if !user?
+				return res.redirect('/restricted')
+			else if userIsOwner user, project || user.isAdmin
 				next()
 			else
 				logger.log user_id: user?._id, email: user?.email, "user is not owner of project redirecting to restricted page"