Merge pull request #83 from Moesif/fix-merge-tags

Fix merge tags

Author: xinghengwang

lib/governanceRulesManager.js

@@ -104,6 +104,8 @@ function recursivelyReplaceValues(tempObjectOrVal, mergeTagValues, ruleVariables
 
       tempString = tempString.replace('{{' + varName + '}}', replacementValue);
     });
+
+    return tempString;
   }
 
   if (isNil(tempObjectOrVal)) {
@@ -142,6 +144,9 @@ function modifyResponseForOneRule(rule, responseHolder, mergeTagValues) {
   }
 
   if (rule.block) {
+    // also need to set this in case it is missing.
+    // blocked body is always json
+    responseHolder.headers['Content-Type'] = 'application/json';
     // in case of rule block, we replace the status and body.
     const ruleResBody = safeGet(rule, 'response.body');
     const replacedBody = recursivelyReplaceValues(ruleResBody, mergeTagValues, ruleVariables);
@@ -424,7 +429,10 @@ GovernanceRulesManager.prototype.applyRuleList = function (
     );
     const mergeTagValues = ruleValuePair && ruleValuePair.values;
     try {
-      return modifyResponseForOneRule(currentRule, prevResponseHolder, mergeTagValues);
+      self.log('modify file response for one rule, prev responseHolder ', { prevResponseHolder, currentRule, mergeTagValues });
+      const resultResponseHolder = modifyResponseForOneRule(currentRule, prevResponseHolder, mergeTagValues);
+      self.log('finished modify response', { resultResponseHolder });
+      return resultResponseHolder;
     } catch (err) {
       self.log('error applying rule ' + currentRule._id + ' ' + err.toString());
       return prevResponseHolder;

lib/outgoingRecorder.js

@@ -123,4 +123,4 @@ function _createOutgoingRecorder(saveEvent, moesifOptions, logger) {
   };
 }
 
-module.exports = _createOutgoingRecorder;
+module.exports = _createOutgoingRecorder;

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "moesif-nodejs",
-  "version": "3.5.3",
+  "version": "3.5.5",
   "description": "Monitoring agent to log API calls to Moesif for deep API analytics",
   "main": "lib/index.js",
   "typings": "dist/index.d.ts",