Merge pull request #234 from apigee-internal/b133195226

Reload crashes EdgeMicro 2.5.35

Author: srinandan

cli/lib/gateway.js

@@ -15,6 +15,7 @@ const defaultPollInterval = 600;
 const uuid = require('uuid/v1');
 const debug = require('debug')('microgateway');
 const jsdiff = require('diff');
+const _ = require('lodash');
 
 const Gateway = function() {};
 
@@ -326,19 +327,22 @@ function hasConfigChanged(oldConfig, newConfig) {
     //do not compare uid
     delete oldConfig['uid'];
 
-    var diff = jsdiff.diffWords(JSON.stringify(oldConfig), JSON.stringify(newConfig));
-    if (diff.length == 1) {
+    
+    if (_.isEqual(oldConfig, newConfig)) {
         debug("no changes detected");
         return false;
     } else {
-        diff.forEach(function(part) {
-            if (part.added)
-                debug("Added->" + part.value);
-            else if (part.removed)
-                debug("Removed->" + part.value);
-            else
-                debug("Unchanged->" + part.value);
-        });
+        if (debug.enabled) {
+            var diff = jsdiff.diffWords(JSON.stringify(oldConfig), JSON.stringify(newConfig));
+            diff.forEach(function(part) {
+                if (part.added)
+                    debug("Added->" + part.value);
+                else if (part.removed)
+                    debug("Removed->" + part.value);
+                else
+                    debug("Unchanged->" + part.value);
+            });
+        }
         return true;
     }
 }

cli/lib/reload-cluster.js

@@ -254,6 +254,8 @@ var ReloadCluster = (file, opt) => {
    */
   self.reload = function (cb) {
     if (!cluster.isMaster) return;
+    //clear the cache before terminating the process
+    cache.clear(function(){});
     respawnerTimers.clear();
 
     function allReady(cb) {

npm-shrinkwrap.json

@@ -1,6 +1,6 @@
 {
   "name": "edgemicro",
-  "version": "2.5.35",
+  "version": "2.5.36",
   "description": "Apigee Edge Microgateway",
   "main": "index.js",
   "license": "Apache-2.0",