Merge pull request #114 from Financial-Times/reliability-kit-logger

Breaking: replace n-logger with Reliability Kit logger
Financial-Times · Aug 8, 2023 · a18fea9 · a18fea9
2 parents b15427b + 93306f6
commit a18fea9
Show file tree

Hide file tree

Showing 6 changed files with 939 additions and 179 deletions.
diff --git a/lib/active.js b/lib/active.js
@@ -2,7 +2,7 @@
 
 const dynamos = require('./dynamos');
 const get = require('./get');
-const logger = require('@financial-times/n-logger').default;
+const logger = require('@dotcom-reliability-kit/logger');
 const surviver = require('./promise-surviver');
 
 const HEALTHCHECK_URL = 'https://www.ft.com/__$HEALTHCHECK';

diff --git a/lib/batch-get.js b/lib/batch-get.js
@@ -2,7 +2,7 @@
 
 const denodeify = require('denodeify');
 const timeoutPromise = require('./timeout');
-const logger = require('@financial-times/n-logger').default;
+const logger = require('@dotcom-reliability-kit/logger');
 const calculateResponseTime = require('./calculate-response-time');
 const encodeDynamoObject = require('./encode-dynamo-object');
 const decodeDynamoObject = require('./decode-dynamo-object');
@@ -47,7 +47,7 @@ module.exports = opts => {
 			});
 	}, timeout)
 		.catch(error => {
-			logger.info({ event: 'URL_MGMT_API_ERROR', error: error.toString() });
+			logger.info({ event: 'URL_MGMT_API_ERROR' }, error);
 			if (error.toString().indexOf('timed out') > -1) {
 				metrics.count(`${table}_status_timeout`);
 			}

diff --git a/lib/get.js b/lib/get.js
@@ -2,7 +2,7 @@
 
 const denodeify = require('denodeify');
 const timeoutPromise = require('./timeout');
-const logger = require('@financial-times/n-logger').default;
+const logger = require('@dotcom-reliability-kit/logger');
 const calculateResponseTime = require('./calculate-response-time');
 const encodeDynamoObject = require('./encode-dynamo-object');
 const decodeDynamoObject = require('./decode-dynamo-object');
@@ -39,7 +39,7 @@ module.exports = opts => {
 			});
 	}, timeout)
 		.catch(error => {
-			logger.info({ event: 'URL_MGMT_API_ERROR', error: error.toString() });
+			logger.info({ event: 'URL_MGMT_API_ERROR' }, error);
 			if (error.toString().indexOf('timed out') > -1) {
 				metrics.count(`${table}_status_timeout`);
 			}