released 1.5.3 with redis connection fixes

Author: kwhitley

.travis.yml

@@ -5,4 +5,5 @@ node_js:
   - 8.0.0
   - 10.0.0
   - 11.0.0
+  - 12.0.0
 after_success: npm run coverage

README.md

@@ -269,10 +269,11 @@ Special thanks to all those that use this library and report issues, but especia
 
 ### Bugfixes, tweaks, documentation, etc.
 
-- @Amhri, @Webcascade, @conmarap, @cjfurelid, @scambier, @lukechilds, @Red-Lv, @gesposito, @viebel, @RowanMeara, @GoingFast, @luin, @keithws, @daveross, @apascal
+- @Amhri, @Webcascade, @conmarap, @cjfurelid, @scambier, @lukechilds, @Red-Lv, @gesposito, @viebel, @RowanMeara, @GoingFast, @luin, @keithws, @daveross, @apascal, @guybrush
 
 ### Changelog
 
+- **v1.5.3** - multiple fixes: Redis should be connected before using (thanks @guybrush)
 - **v1.5.2** - multiple fixes: Buffer deprecation and \_headers deprecation, { trackPerformance: false } by default per discussion (sorry semver...)
 - **v1.5.1** - adds { trackPerformance } option to enable/disable performance tracking (thanks @fernandolguevara)
 - **v1.5.0** - exposes apicache.getPerformance() for per-route cache metrics (@killdash9 continues to deliver)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "apicache",
-  "version": "1.5.2",
+  "version": "1.5.3",
   "scripts": {
     "lint": "eslint .",
     "pretest": "npm run lint",
@@ -10,7 +10,7 @@
     "prepublish": "npm run test"
   },
   "engines": {
-    "node": ">=6.2.0"
+    "node": ">=6.2.0 <13"
   },
   "description": "An ultra-simplified API response caching middleware for Express/Node using plain-english durations.",
   "main": "./src/apicache.js",
@@ -44,13 +44,13 @@
     "express": "^4.17.1",
     "fakeredis": "^2.0.0",
     "husky": "^3.0.4",
-    "mocha": "^5.2.0",
+    "mocha": "^7.0.0",
     "nyc": "^13.3.0",
     "prettier": "^1.18.2",
     "pretty-quick": "^1.11.1",
     "restify": "^7.7.0",
     "restify-etag-cache": "^1.0.12",
-    "supertest": "^3.4.2"
+    "supertest": "^4.0.2"
   },
   "dependencies": {},
   "husky": {

src/apicache.js

@@ -132,7 +132,7 @@ function ApiCache() {
     var redis = globalOptions.redisClient
     var expireCallback = globalOptions.events.expire
 
-    if (redis) {
+    if (redis && redis.connected) {
       try {
         redis.hset(key, 'response', JSON.stringify(value))
         redis.hset(key, 'duration', duration)