guard redis error handler (#18)

ibm-cloud-solutions · Sep 9, 2016 · 590d1e8 · 590d1e8
1 parent 26185b4
commit 590d1e8
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 20 deletions.
diff --git a/src/scripts/redis.delete.js b/src/scripts/redis.delete.js
@@ -39,15 +39,18 @@ let currentResponse;
 
 module.exports = (robot) => {
 
-	redis.on('error', function(err) {
-		if (currentResponse) {
-			let message = i18n.__('redis.error', err);
-			robot.emit('ibmcloud.formatter', {
-				response: currentResponse,
-				message: message
-			});
-		}
-	});
+	if (redis) {
+		redis.on('error', function(err) {
+			if (currentResponse) {
+				let message = i18n.__('redis.error', err);
+				robot.emit('ibmcloud.formatter', {
+					response: currentResponse,
+					message: message
+				});
+			}
+		});
+	}
+
 	// Natural Language match
 	robot.on(DELETE_ID, (res) => {
 		robot.logger.debug(`${TAG}: ${DELETE_ID} - Natural Language match - res.message.text=${res.message.text}.`);

diff --git a/src/scripts/redis.nottls.js b/src/scripts/redis.nottls.js
@@ -119,8 +119,6 @@ module.exports = (robot) => {
 
 	function processMonitorNoTtlsWrapper(res) {
 		if (currentMonitor) {
-			// return, I'm already monitoring so I can't help you
-			// this is a stopgap until we can get true multi-user support in place
 			let warning = i18n.__('monitor.ttls.busy');
 			robot.emit('ibmcloud.formatter', {
 				response: res,

diff --git a/src/scripts/redis.slowlog.js b/src/scripts/redis.slowlog.js
@@ -36,15 +36,17 @@ let lastResponse;
 
 module.exports = (robot) => {
 
-	redis.on('error', function(err) {
-		if (lastResponse) {
-			let message = i18n.__('redis.error', err);
-			robot.emit('ibmcloud.formatter', {
-				response: lastResponse,
-				message: message
-			});
-		}
-	});
+	if (redis) {
+		redis.on('error', function(err) {
+			if (lastResponse) {
+				let message = i18n.__('redis.error', err);
+				robot.emit('ibmcloud.formatter', {
+					response: lastResponse,
+					message: message
+				});
+			}
+		});
+	}
 
 	// Natural Language match
 	robot.on(SLOWLOG_ID, (res) => {