@josecarlosrz/logger - npm Package Compare versions

Comparing version 1.0.16 to 1.0.17

index.js

@@ -11,16 +11,40 @@ const nodeConsole = require('console');
 
-const Logger = {
+const LoggerChecker = {
     
+    isLoggerDisabled: () => {
+        
+        return (
+            process.env.hasOwnProperty('LOGGER_LOGGER_DISABLED')
+            && process.env.LOGGER_LOGGER_DISABLED
+        );
+        
+    },
+    
+    isSleepDisabled: () => {
+        
+        return (
+            process.env.hasOwnProperty('LOGGER_SLEEP_DISABLED')
+            && process.env.LOGGER_SLEEP_DISABLED
+        );
+        
+    },
+    
+    isDev: () => {
+        
+        return (
+            !process.env.hasOwnProperty('NODE_ENV')
+            || process.env.NODE_ENV === 'development'
+        );
+        
+    },
+    
+};
+
+const TmpLogger = {
+    
     logger: function(message, ...args) {
         
-        if (
-            process.env.hasOwnProperty('DISABLE_LOGGER_LOGGER')
-            && process.env.DISABLE_LOGGER_LOGGER
-        ) {
-            return;
-        }
-        
         const stack = stackTrace.get();
         
-        const caller = stack[1];
+        const caller = stack[2];
         
@@ -34,2 +58,3 @@ const file = caller.getFileName();
         console.log('--');
+        console.log();
         
@@ -44,6 +69,4 @@ console.log('[Logger]');
             
-            console.log('[Args]');
+            console.group('[Args]');
             
-            console.group();
-            
             args.forEach(arg => console.log(arg));
@@ -55,2 +78,3 @@
         
+        console.log();
         console.log('--');
@@ -63,15 +87,3 @@
         return new Promise((resolve, reject) => {
-            
-            if (
-                process.env.hasOwnProperty('DISABLE_LOGGER_SLEEP')
-                && process.env.DISABLE_LOGGER_SLEEP
-            ) {
-                
-                resolve();
-                return;
-                
-            }
-            
             setTimeout(resolve, seconds * 1000);
-            
         });
@@ -81,6 +93,24 @@
     
+};
+
+const Logger = {
+    
+    logger: function(message, ...args) {
+        
+        if (!LoggerChecker.isLoggerDisabled())
+            TmpLogger.logger(message, ...args);
+        
+    },
+    
+    sleep: async function(seconds) {
+        
+        if (!LoggerChecker.isSleepDisabled())
+            await TmpLogger.sleep(seconds);
+        
+    },
+    
     devLogger: function(message, ...args) {
         
-        if (process.env.NODE_ENV === 'development')
-            Logger.logger(message, ...args);
+        if (!LoggerChecker.isLoggerDisabled() && LoggerCheck.isDev())
+            TmpLogger.logger(message, ...args);
         
@@ -91,4 +121,4 @@ },
         
-        if (process.env.NODE_ENV === 'development')
-            await Logger.sleep(seconds);
+        if (!LoggerChecker.isSleepDisabled() && LoggerCheck.isDev())
+            await TmpLogger.sleep(seconds);
         
@@ -95,0 +125,0 @@ },

package.json

 {
   "name": "@josecarlosrz/logger",
-  "version": "1.0.16",
+  "version": "1.0.17",
   "description": "Function to log each component of our application",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 2 instances in 1 package

Fixed alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 2 instances in 1 package

Improved metrics

Total package byte prevSize

3274

increased by21.57%

Lines of code

71

increased by33.96%

No dependency changes