connect-roles - npm Package Compare versions

Comparing version 0.1.1 to 0.1.2

index.js

 var functionList = [];
 var failureHandler = function (req, res, action){
-	res.send(403);
+    res.send(403);
 };
 var defaultUser = {};
 module.exports = function middleware(req, res, next){
-  var oldUser = req.user;
-  req.user = req.user || Object.create(defaultUser);
-  if(oldUser){
-    req.user.isAuthenticated = true;
-  }else{
-    req.user.isAuthenticated = false;
-  }
-	if(req.user){
-		req.user.is = tester(req);
-		req.user.can = tester(req);
-	}
-	next();
+    var oldUser = req.user;
+    req.user = req.user || Object.create(defaultUser);
+    if(oldUser){
+        req.user.isAuthenticated = true;
+    }else{
+        req.user.isAuthenticated = false;
+    }
+    if(req.user){
+        req.user.is = tester(req);
+        req.user.can = tester(req);
+    }
+    next();
 };
 
 
+module.exports.log = false;
 
-
 module.exports.can = routeTester;
 module.exports.is = routeTester;
 module.exports.isAuthenticated = function(req,res,next){
-	if(arguments.length === 0) return module.exports.isAuthenticated;
-	if (req.user && req.user.isAuthenticated) next();
-	else if(req.user) failureHandler(req, res, "isAuthenticated");
-	else throw "Request.user was null or undefined, include middleware";
+    if(arguments.length === 0) return module.exports.isAuthenticated;
+    if (req.user && req.user.isAuthenticated) next();
+    else if(req.user) failureHandler(req, res, "isAuthenticated");
+    else throw "Request.user was null or undefined, include middleware";
 };
@@ -35,19 +35,18 @@
 module.exports.useAuthorizationStrategy = function(path, fn){
-	if(typeof path === "function"){
-		fn = path
-	}
-	functionList.push(function(user, action, stop){
-		if(typeof path === "string" && path != action){
-			return null;
-		}
-		
-		return fn.call(this, user, action, stop);
-	});
-	return this;
+    if(typeof path === "function"){
+        fn = path
+    }
+    functionList.push(function(user, action, stop){
+            if(typeof path === "string" && path != action){
+                return null;
+            }
+            return fn.call(this, user, action, stop);
+    });
+    return this;
 };
 module.exports.setFailureHandler = function(fn){
-	failureHandler = fn;
+    failureHandler = fn;
 };
 module.exports.setDefaultUser = function(user){
-  defaultUser = user;
+    defaultUser = user;
 };
@@ -57,31 +56,31 @@
 function tester(req){
-	return function(action){
-		var result = null,
-			vote;
-		var stop = false;
-		function stopNow(vote){
-			stop = true;
-        	if (vote === false) result = false;
-        	else if (vote === true) result = true;
-
-		}
-    	for (var i = 0; i<functionList.length && !stop; i++){
-    		var fn = functionList[i];
-        	vote = fn.call(req, req.user, action, stopNow);
-        	if(vote === false) return false;
-        	else if (vote === true) result = true;
-    	}
-    	return (result === true);
-	}
+    return function(action){
+        var result = null,
+        vote;
+        var stop = false;
+        function stopNow(vote){
+            stop = true;
+            if (vote === false) result = false;
+            else if (vote === true) result = true;
+        }
+        for (var i = 0; i<functionList.length && !stop; i++){
+            var fn = functionList[i];
+            vote = fn.call(req, req.user, action, stopNow);
+            if(vote === false) return false;
+            else if (vote === true) result = true;
+        }
+        if(module.exports.log) console.log('Check Permission: ' + (user.id||user.name||user) + ".can('" + action + "'') -> " + (result === true));
+        return (result === true);
+    }
 }
 
-function routeTester(action){		
-	return function(req,res,next){
-		if(tester(req)(action)){
-			next();
-		}else{
-			//Failed authentication.
-			failureHandler(req, res, action);		
-		}
-	};
+function routeTester(action){    
+    return function(req,res,next){
+        if(tester(req)(action)){
+            next();
+        }else{
+            //Failed authentication.
+            failureHandler(req, res, action);    
+        }
+    };
 };

package.json

 {
 	"name": "connect-roles",
 	"description": "Provides dynamic roles based authentication for node.js connect and express servers.",
-	"version": "0.1.1",
+	"version": "0.1.2",
 	"homepage": "http://documentup.com/Tuskan360/connect-roles",
@@ -6,0 +6,0 @@ "repository": {

No alert changes

Improved metrics

Total package byte prevSize

13277

increased by3.27%

Lines of code

171

increased by1.18%

No dependency changes