Fix issue where admin bus crashes when "db" or "pass" is not specifie…

…d in redis config
balderdashy · Jan 15, 2016 · 14210dc · 14210dc
1 parent 410806c
commit 14210dc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/connect-to-admin-bus.js b/lib/connect-to-admin-bus.js
@@ -18,6 +18,9 @@ module.exports = function (app) {
 
   return function connectToAdminBus(adapterDef, cb){
 
+    // Indicate that this connection is for the admin bus
+    adapterDef.adminBus = true;
+
     // Prepare the driver for the admin bus (e.g. Redis)
     prepareDriver(adapterDef, function(err) {
       if (err) {return cb(err);}

diff --git a/lib/prepare-driver.js b/lib/prepare-driver.js
@@ -22,7 +22,7 @@ module.exports = function (app){
     if (adapterModuleName === 'socket.io-redis') {
 
       // Create raw redis clients if necessary
-      var rawClientsNecessary = adapterConfig.pass || adapterConfig.db;
+      var rawClientsNecessary = adapterConfig.pass || adapterConfig.db || adapterDef.adminBus;
       if (!rawClientsNecessary) {
         return cb();
       }