added support for overriding redis config

Created by: trevorah

Side affect: the internal redis-client signature changes from create({ config, logger }) to create(options, logger). This is because config is no longer an nconf instance, its now a map, and so it seems a bit silly to put the options map in another map.

lib/env.js

@@ -60,13 +60,16 @@ exports.create = function(configDirectory) {
       getClient: function() {
         if(mainRedisClient) return mainRedisClient;
-        mainRedisClient = redisClient.create({ config: config, logger: logger });
+        // no options override for singleton
+        var options = config.get('redis');
+        mainRedisClient = redisClient.create(options, logger);
         return mainRedisClient;
       },
-      createClient: function() {
-        return redisClient.create({ config: config, logger: logger });
+      createClient: function(options) {
+        options = options || config.get('redis');
+        return redisClient.create(options, logger);
       },
       /**
@@ -74,7 +77,10 @@ exports.create = function(configDirectory) {
        */
       createTransientClient: function(callback) {
         var mainClient = this.getClient();
-        return redisClient.createTransientClient(mainClient, { config: config, logger: logger }, callback);
+
+        // no options override for singleton
+        var options = config.get('redis');
+        return redisClient.createTransientClient(mainClient, options, logger, callback);
       },
       quitClient: function(client) {