camshaft - npm Package Compare versions

Comparing version 0.66.0 to 0.67.0

CHANGELOG.md

 # Changelog
 
+## 0.67.0
+Release 2020-07-22
+
+- Chore:
+  - Adapt logger to the standard JSON sctructure
+
 ## 0.66.0
@@ -4,0 +10,0 @@ Release 2020-06-22

lib/analysis.js

@@ -106,3 +106,3 @@ 'use strict';
             function analysis$track (analysis, done) {
-                logger.info({ analysis });
+                logger.info({ analysis }, 'Analysis created');
 
@@ -109,0 +109,0 @@ databaseService.trackAnalysis(analysis, function (err) {

lib/logging/logger.js

@@ -20,2 +20,12 @@ 'use strict';
             level: LOG_LEVEL || logLevelFromNodeEnv,
+            formatters: {
+                level (label) {
+                    if (label === 'warn') {
+                        return { levelname: 'warning' };
+                    }
+                    return { levelname: label };
+                }
+            },
+            messageKey: 'message',
+            timestamp: () => `,"timestamp":"${new Date(Date.now()).toISOString()}"`,
             serializers: {
@@ -38,3 +48,3 @@ analysis: (analysis) => {
                 },
-                error: (error) => Array.isArray(error) ? error.map((err) => errorSerializer(err)) : [errorSerializer(error)]
+                exception: errorSerializer
             }
@@ -41,0 +51,0 @@ };

lib/node/limits.js

@@ -13,3 +13,3 @@ // Functions to apply limits to nodes
         err.node = { id: node.id(), ...node.toJSON() };
-        logger.warn({ error: err });
+        logger.warn({ exception: err }, err.message);
         return callback(null);
@@ -20,5 +20,4 @@ }
             // if estimation is not available don't abort the analysis
-            err.message = `Number of rows estimation not available. ${err.message}`;
             err.node = { id: node.id(), ...node.toJSON() };
-            logger.warn({ error: err });
+            logger.warn({ exception: err }, 'Number of rows estimation not available');
             err = null;
@@ -40,3 +39,3 @@ } else {
         err.node = { id: node.id(), ...node.toJSON() };
-        logger.warn({ error: err });
+        logger.warn({ exception: err }, err.message);
         return callback(null);
@@ -46,5 +45,4 @@ }
         if (err) {
-            err.message = `Number of rows estimation not available. ${err.message}`;
             err.node = { id: node.id(), ...node.toJSON() };
-            logger.warn({ error: err });
+            logger.warn({ exception: err }, 'Number of rows estimation not available');
             err = null;
@@ -69,3 +67,3 @@ } else {
         err.node = { id: node.id(), ...node.toJSON() };
-        logger.warn({ error: err });
+        logger.warn({ exception: err }, err.message);
         return callback(null);
@@ -75,3 +73,3 @@ } else if (groupedLimitPresent && !_verifyCategoryData(categoriesData)) {
         err.node = { id: node.id(), ...node.toJSON() };
-        logger.warn({ error: err });
+        logger.warn({ exception: err }, err.message);
         return callback(null);
@@ -101,3 +99,3 @@ }
         err.node = { id: node.id(), ...node.toJSON() };
-        logger.warn({ error: err });
+        logger.warn({ exception: err }, err.message);
         return callback(null);
@@ -107,3 +105,3 @@ } else if (!_verifyCategoryData(categoryTarget)) {
         err.node = { id: node.id(), ...node.toJSON() };
-        logger.warn({ error: err });
+        logger.warn({ exception: err }, err.message);
         return callback(null);
@@ -129,5 +127,4 @@ }
             // something went wrong... lack of stats? anyway, we'll let the analysis pass
-            err.message = `Number of rows estimation not available. ${err.message}`;
             err.node = { id: node.id(), ...node.toJSON() };
-            logger.warn({ error: err });
+            logger.warn({ exception: err }, 'Number of rows estimation not available');
             return callback(null);
@@ -141,5 +138,4 @@ } else {
                         if (err) {
-                            err.message = `Number of distinct values not available. ${err.message}`;
                             err.node = { id: node.id(), ...node.toJSON() };
-                            logger.warn({ error: err });
+                            logger.warn({ exception: err }, 'Number of distinct values not available');
                         } else {
@@ -146,0 +142,0 @@ numCategories = num;

package.json

 {
   "name": "camshaft",
-  "version": "0.66.0",
+  "version": "0.67.0",
   "description": "Analysis library to create data views from queries",
@@ -5,0 +5,0 @@ "keywords": [

No alert changes

Improved metrics

Total package byte prevSize

1312017

increased by0.03%

Lines of code

30029

increased by0.02%

No dependency changes