Merge pull request #128 from antoineleclair/redis-port-fix

Fix #116, --redis-port was ignored when no host specified
joeferner · Mar 22, 2014 · a7028a1 · a7028a1
2 parents da1ee16 + 19e0a53
commit a7028a1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/bin/redis-commander.js b/bin/redis-commander.js
@@ -74,7 +74,7 @@ myUtils.getConfig(function (err, config) {
       console.log(err);
       process.exit();
     }
-    if (args['redis-host'] || args['redis-socket']) {
+    if (args['redis-host'] || args['redis-port'] || args['redis-socket']) {
       var db = parseInt(args['redis-db']);
       if (db == null || isNaN(db)) {
         db = 0
@@ -85,6 +85,7 @@ myUtils.getConfig(function (err, config) {
         "password": args['redis-password'] || "",
         "dbIndex": db
       };
+
       if (!myUtils.containsConnection(config.default_connections, newDefault)) {
         redisConnections.push(redis.createClient(newDefault.port, newDefault.host));
         if (args['redis-password']) {