no message

salesforce · Aug 19, 2017 · 1dd0b51 · 1dd0b51
1 parent aa845b4
commit 1dd0b51
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 10 deletions.
diff --git a/cache/redisCache.js b/cache/redisCache.js
@@ -15,6 +15,7 @@
 const redis = require('redis');
 const logger = require('winston');
 const rconf = require('../config').redis;
+const featureToggles = require('feature-toggles');
 
 /*
  * This will add "...Async" to all node_redis functions (e.g. return
@@ -77,22 +78,24 @@ const client = {
 };
 
 Object.keys(client).forEach((key) => {
-  client[key].on('connect', () => {
-    logger.info(`redisClientConnection=${key} event=connect`);
-  });
-
   client[key].on('error', (err) => {
     logger.error(`redisClientConnection=${key} event=error`, err);
     return new Error(err);
   });
 
-  client[key].on('ready', () => {
-    logger.info(`redisClientConnection=${key} event=ready`);
-  });
+  if (featureToggles.isFeatureEnabled('enableRedisConnectionLogging')) {
+    client[key].on('connect', () => {
+      logger.info(`redisClientConnection=${key} event=connect`);
+    });
 
-  client[key].on('reconnecting', () => {
-    logger.info(`redisClientConnection=${key} event=reconnecting`);
-  });
+    client[key].on('ready', () => {
+      logger.info(`redisClientConnection=${key} event=ready`);
+    });
+
+    client[key].on('reconnecting', () => {
+      logger.info(`redisClientConnection=${key} event=reconnecting`);
+    });
+  }
 });
 
 module.exports = {

diff --git a/config/toggles.js b/config/toggles.js
@@ -68,6 +68,10 @@ const longTermToggles = {
   enableRealtimeActivityLogs:
     environmentVariableTrue(pe, 'ENABLE_REALTIME_ACTIVITY_LOGS'),
 
+  // Enable redis client connection logging.
+  enableRedisConnectionLogging: environmentVariableTrue(pe,
+    'ENABLE_REDIS_CONNECTION_LOGGING'),
+
   // Use redis sampleStore with postgres db backup.
   enableRedisSampleStore:
     environmentVariableTrue(pe, 'ENABLE_REDIS_SAMPLE_STORE'),