Fix connection errors #10

AbdullahAli · 2018-10-17T11:37:21Z

Fix connection errors by adding redis reconnection strategy a5896c6 see https://github.com/luin/ioredis#auto-reconnect
Add logger context as email-stethoscope to differentiate logs c39d488

andreaslof

leafrogers · 2018-10-17T11:43:45Z

lib/utils/logger.js

@@ -0,0 +1,7 @@
+const logger = require('@financial-times/n-logger').default;
+
+logger.addContext({


leafrogers

🍗🍗🍗

AbdullahAli added 2 commits October 17, 2018 12:33

Add reconnection logic. 🐿 v2.10.3

a5896c6

Add logger context. 🐿 v2.10.3

c39d488

AbdullahAli requested a review from a team October 17, 2018 11:37

andreaslof approved these changes Oct 17, 2018

View reviewed changes

leafrogers reviewed Oct 17, 2018

View reviewed changes

lib/utils/logger.js

@@ -0,0 +1,7 @@

const logger = require('@financial-times/n-logger').default;

logger.addContext({

Copy link

Contributor

leafrogers Oct 17, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

Bump version. 🐿 v2.10.3

4be14ab

leafrogers approved these changes Oct 17, 2018

View reviewed changes

AbdullahAli merged commit b65a360 into master Oct 17, 2018

AbdullahAli deleted the fix-connection-errors branch October 17, 2018 11:46

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix connection errors #10

Fix connection errors #10

AbdullahAli commented Oct 17, 2018 •

edited

andreaslof left a comment

leafrogers Oct 17, 2018

leafrogers left a comment

		@@ -0,0 +1,7 @@
		const logger = require('@financial-times/n-logger').default;

		logger.addContext({

Fix connection errors #10

Fix connection errors #10

Conversation

AbdullahAli commented Oct 17, 2018 • edited

andreaslof left a comment

Choose a reason for hiding this comment

leafrogers Oct 17, 2018

Choose a reason for hiding this comment

leafrogers left a comment

Choose a reason for hiding this comment

AbdullahAli commented Oct 17, 2018 •

edited