feat(config): Merge with existing config when writing to memcache. (#151

) r=vladikoff

This makes it easier to change a single setting while leaving all
the others alone.

package.json

@@ -28,6 +28,7 @@
     "deep-equal": "1.0.1",
     "ip": "1.1.3",
     "ip-reputation-js-client": "1.0.1",
+    "lodash.merge": "4.6.0",
     "memcached": "2.2.1",
     "newrelic": "1.30.1",
     "restify": "4.1.1"

scripts/config-set.js

@@ -22,6 +22,7 @@
 var Memcached = require('memcached')
 var P = require('bluebird')
 var BL = require('bl')
+var merge = require('lodash.merge')
 
 P.promisifyAll(Memcached.prototype)
 
@@ -35,6 +36,17 @@ if (!/.+:\d+/.test(process.argv[2])) {
   return console.error(x)
 }
 
+function writeMergedSettings(mc, key, newSettings) {
+  return mc.getAsync(key)
+    .then(
+      function (settings) {
+        settings = settings || {}
+        merge(settings, newSettings)
+        return mc.setAsync(key, settings, 0)
+      }
+    )
+}
+
 var b = process.stdin.pipe(new BL())
 
 process.stdin.on('end', function () {
@@ -49,19 +61,19 @@ process.stdin.on('end', function () {
 
   var actions = []
   if (input.limits) {
-    actions.push(mc.setAsync('limits', input.limits, 0))
+    actions.push(writeMergedSettings(mc, 'limits', input.limits))
   }
 
   if (input.allowedIPs) {
-    actions.push(mc.setAsync('allowedIPs', input.allowedIPs, 0))
+    actions.push(writeMergedSettings(mc, 'allowedIPs', input.allowedIPs))
   }
 
   if (input.allowedEmailDomains) {
-    actions.push(mc.setAsync('allowedEmailDomains', input.allowedEmailDomains, 0))
+    actions.push(writeMergedSettings(mc, 'allowedEmailDomains', input.allowedEmailDomains))
   }
 
   if (input.requestChecks) {
-    actions.push(mc.setAsync('requestChecks', input.requestChecks, 0))
+    actions.push(writeMergedSettings(mc, 'requestChecks', input.requestChecks))
   }
 
   P.all(actions)