Merge pull request #425 from CodeNow/only-cache-inspect-state

mongo doesn't like dotted keys

Author: tjmehta

lib/models/mongo/instance.js

@@ -19,6 +19,7 @@ var Boom = require('dat-middleware').Boom;
 var keypather = require('keypather')();
 var isFunction = require('101/is-function');
 var createCount = require('callback-count');
+var removeDottedKeys = require('remove-dotted-keys');
 
 var InstanceSchema = require('models/mongo/schemas/instance');
 
@@ -159,7 +160,6 @@ InstanceSchema.methods.populateModels = function (cb) {
   }
 };
 
-
 /**
  * findAndModify container with containerInpect data (and dockerHost)
  * @param  {Object}   containerInspect docker
@@ -172,7 +172,7 @@ InstanceSchema.methods.modifySetContainer = function (containerInspect, dockerHo
   var updateFields = { // set it on the model, mongoose can cast things
     'container.dockerHost'     : dockerHost,
     'container.dockerContainer': info.Id,
-    'container.inspect': info
+    'container.inspect'        : removeDottedKeys(info)
   };
   // don't override ports if they are undefined so that hosts can be cleaned up
   var ports = info.NetworkSettings && info.NetworkSettings.Ports;

lib/remove-dotted-keys.js

@@ -0,0 +1,21 @@
+'use strict';
+var isObject = require('101/is-object');
+
+// modifies original object
+module.exports = function removeDottedKeys (obj) {
+  if (isObject(obj)) {
+    try {
+      Object.keys(obj).forEach(function (key) {
+        if (~key.indexOf('.')) {
+          delete obj[key];
+        }
+        removeDottedKeys(obj[key]);
+      });
+    }
+    catch (err) {
+      // object.keys on non-object
+      // just skip it.
+    }
+  }
+  return obj;
+};