Merge pull request #23 from gechols/patch-1

Added error check
jpodwys · Jul 1, 2020 · ff44f83 · ff44f83
2 parents 2cba5e6 + 6f10ceb
commit ff44f83
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/redisCacheModule.js b/redisCacheModule.js
@@ -173,16 +173,18 @@ function redisCacheModule(config){
     var cacheKeys = keys.map(prefixKey);
     self.db.mget(cacheKeys, function (err, response){
       var obj = {};
-      for(var i = 0; i < response.length; i++){
-        if(response[i] !== null){
-          try {
-            response[i] = self.JSON.parse(response[i]);
-          } catch (err) {
-            if(self.logJsonParseFailures) {
-              log(true, 'Error parsing JSON, err:', err);
+      if(!err && response) {
+        for(var i = 0; i < response.length; i++){
+          if(response[i] !== null){
+            try {
+              response[i] = self.JSON.parse(response[i]);
+            } catch (err) {
+              if(self.logJsonParseFailures) {
+                log(true, 'Error parsing JSON, err:', err);
+              }
             }
+            obj[keys[i]] = response[i];
           }
-          obj[keys[i]] = response[i];
         }
       }
       cb(err, obj, index);
@@ -396,4 +398,4 @@ function redisCacheModule(config){
 }
 
 module.exports = redisCacheModule;
-redisCacheModule._retryStrategy = retryStrategy;
+redisCacheModule._retryStrategy = retryStrategy;