@contrast/agent 4.4.1 → 4.6.0

package/bin/VERSION

@@ -1 +1 @@
-2.26.0
+2.27.3

package/lib/assess/hapi/route-coverage.js

@@ -151,9 +151,9 @@ class RouteCoverage {
    */
   createSignature({ method, path }) {
     let signature = 'server.route({ method: ';
-    Array.isArray(method)
-      ? (signature += `["${method.join('", "')}"]`)
-      : (signature += `"${method}"`);
+    signature += Array.isArray(method)
+      ? `["${method.join('", "')}"]`
+      : `"${method}"`;
 
     signature += `, path: "${path}" })`;
     return signature;

package/lib/assess/membrane/index.js

@@ -280,10 +280,6 @@ class Membrane {
       return this.wrapArray(target, metadata);
     }
 
-    // Object.defineProperty(target, util.inspect.custom, {
-    //   target: typeof target === 'string' ? () => `'${target}'` : () => target
-    // });
-
     return this.wrapObject(target, metadata);
   }
 }
@@ -310,12 +306,10 @@ function makeHandler(membrane, metadata) {
       // https://www.ecma-international.org/ecma-262/7.0/#sec-proxy-object-internal-methods-and-internal-slots-get-p-receiver
       // satisfy invariant
       const desc = Object.getOwnPropertyDescriptor(tar, prop);
-      // if (desc && (desc.writable || desc.set || desc.configurable)) {
       if (desc && desc.configurable) {
         r = membrane.wrap(r, copyMetadata(tar, prop, metadata));
-      } else {
-        // invariant case; can't wrap
-      }
+      } // else: invariant case; can't wrap
+
       return r;
     },
 

package/lib/assess/membrane/source-membrane.js

@@ -294,10 +294,9 @@ module.exports = class SourceMembrane extends Membrane {
     if (!(metadata.sourceType && metadata.path)) {
       return false;
     }
-    const koaQueryString = metadata.path.match(/(\w+)=/);
-    if (koaQueryString) {
-      // get 1st capture group, fall back to `metadata.path` if for some reason this does not exist
-      metadata.path = koaQueryString[1] || metadata.path;
+    const koaQueryString = metadata.path.split('=');
+    if (koaQueryString[1]) {
+      metadata.path = koaQueryString[0];
     }
     return true;
   }

package/lib/assess/models/base-event.js

@@ -166,7 +166,7 @@ class BaseEvent {
  * @return {BaseEvent[]} sorted list of events
  */
 function sortEvents(events) {
-  const sorted = events.sort((a, b) => {
+  events.sort((a, b) => {
     let parentsfactor = 0;
     if (a.parents.length > b.parents.length) {
       parentsfactor = -1;
@@ -184,7 +184,7 @@ function sortEvents(events) {
     return parentsfactor + timefactor;
   });
 
-  return sorted;
+  return events;
 }
 
 module.exports = BaseEvent;

package/lib/assess/models/call-context.js

@@ -153,9 +153,6 @@ module.exports = class CallContext {
       return value.toString();
     }
 
-    // FIXME
-    // if value === JSON, if value === Buffer, etc to put proper constructor name for static methods?
-
     const constructorName = _.get(value, 'constructor.name', 'null');
 
     if (constructorName === 'Object' && value) {

package/lib/assess/policy/propagators.json

@@ -67,6 +67,26 @@
         "type": "keep"
       }
     },
+    "dust.escapeHtml": {
+      "enabled": true,
+      "source": "P",
+      "target": "R",
+      "tags": ["html-encoded"],
+      "type": "overload",
+      "command": {
+        "type": "keep"
+      }
+    },
+    "dust.escapeJs": {
+      "enabled": true,
+      "source": "P",
+      "target": "R",
+      "tags": ["javascript-encoded"],
+      "type": "overload",
+      "command": {
+        "type": "keep"
+      }
+    },
     "pug.compile": {
       "enabled": true,
       "provider": "./propagators/pug-compile.js"