Only call logger if a logger is actually defined.

OptimalBits · Nov 22, 2011 · 0767fc2 · 0767fc2
1 parent e2b582d
commit 0767fc2
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/acl.js b/lib/acl.js
@@ -530,19 +530,19 @@ Acl.prototype.middleware = function(numPathComponents, userId, actions){
       actions = req.method.toLowerCase();
     }
 
-    var logMessage = 'Requesting '+actions+' on '+resource+' by user '+_userId;
-    logger.debug(logMessage)
+    logger?logger.debug('Requesting '+actions+' on '+resource+' by user '+_userId):null;
+
     acl.isAllowed(_userId, resource, actions, function(err, allowed){
       if (err){
         next(new Error('Error checking permissions to access resource'));
       }else if(allowed === false){
-        logger.debug('Not allowed '+actions+' on '+resource+' by user '+_userId);
+        logger?logger.debug('Not allowed '+actions+' on '+resource+' by user '+_userId):null;
         acl.allowedPermissions(_userId, resource, function(err, obj){
-          logger.debug('Allowed permissions: '+obj)
+          logger?logger.debug('Allowed permissions: '+util.inspect(obj)):null;
         });
         next(new HttpError(401,'Insufficient permissions to access resource'));
       }else{
-        logger.debug('Allowed '+actions+' on '+resource+' by user '+_userId);
+        logger?logger.debug('Allowed '+actions+' on '+resource+' by user '+_userId):null;
         next();
       }
     });