newrelic 7.5.2 → 8.3.0

package/lib/instrumentation/promise.js

@@ -9,7 +9,6 @@ var util = require('util')
 var properties = require('../util/properties')
 var shimmer = require('../shimmer')
 
-
 /**
  * @namespace Library.Spec
  *
@@ -84,6 +83,7 @@ var shimmer = require('../shimmer')
  * @param {function}      library - The promise library.
  * @param {?Library.Spec} spec    - Spec for this promise library mapping.
  */
+/* eslint-disable camelcase */
 module.exports = function initialize(agent, library, spec) {
   if (spec.useFinally == null) {
     spec.useFinally = true
@@ -189,7 +189,7 @@ module.exports = function initialize(agent, library, spec) {
     // implementation uses promise.constructor to create new Promises for
     // calls to `then`, `chain` and `catch` which allows these Promises to
     // also be instrumented.
-    promise.__proto__ = wrappedPromise.prototype  // eslint-disable-line no-proto
+    promise.__proto__ = wrappedPromise.prototype // eslint-disable-line no-proto
 
     return promise
   }
@@ -200,7 +200,7 @@ module.exports = function initialize(agent, library, spec) {
       return
     }
 
-    name = name || (spec.constructor + '.prototype')
+    name = name || spec.constructor + '.prototype'
 
     // Wrap up instance methods.
     _safeWrap(PromiseProto, name, spec.$proto.executor, wrapExecutorCaller)
@@ -340,9 +340,9 @@ module.exports = function initialize(agent, library, spec) {
 
       function wrapHandler(fn, i, arr) {
         if (
-          typeof fn !== 'function' ||               // Not a function
-          fn.name === '__NR_wrappedThenHandler' ||  // Already wrapped
-          (!useAllParams && i !== (arr.length - 1)) // Don't want all and not last
+          typeof fn !== 'function' || // Not a function
+          fn.name === '__NR_wrappedThenHandler' || // Already wrapped
+          (!useAllParams && i !== arr.length - 1) // Don't want all and not last
         ) {
           isWrapped = fn && fn.name === '__NR_wrappedThenHandler'
           return fn
@@ -413,8 +413,7 @@ module.exports = function initialize(agent, library, spec) {
       return __NR_wrappedPromisified
       function __NR_wrappedPromisified() {
         var segment = _createSegment(WRAP_SEGMENT_NAME)
-        var prom = agent.tracer.bindFunction(promisified, segment, true)
-          .apply(this, arguments)
+        var prom = agent.tracer.bindFunction(promisified, segment, true).apply(this, arguments)
 
         if (segment) {
           Contextualizer.link(null, prom, segment, spec.useFinally)
@@ -428,7 +427,7 @@ module.exports = function initialize(agent, library, spec) {
   function _createSegment(name, parent) {
     return agent.config.feature_flag.promise_segments === true
       ? agent.tracer.createSegment(name, null, parent)
-      : (parent || agent.tracer.getSegment())
+      : parent || agent.tracer.getSegment()
   }
 }
 
@@ -543,7 +542,7 @@ Contextualizer.prototype.getSegment = function getSegment() {
 }
 
 Contextualizer.prototype.setSegment = function setSegment(segment) {
-  return this.context.segments[this.idx] = segment
+  return (this.context.segments[this.idx] = segment)
 }
 
 Contextualizer.prototype.toJSON = function toJSON() {
@@ -564,15 +563,19 @@ Contextualizer.prototype.continue = function continueContext(prom) {
 
   // No `finally` means we need to hook into resolve and reject individually and
   // pass through whatever happened.
-  return prom.then(function __NR_thenContext(val) {
-    __NR_continueContext()
-    return val
-  }, function __NR_catchContext(err) {
-    __NR_continueContext()
-    throw err // Re-throwing promise rejection, this is not New Relic's error.
-  })
+  return prom.then(
+    function __NR_thenContext(val) {
+      __NR_continueContext()
+      return val
+    },
+    function __NR_catchContext(err) {
+      __NR_continueContext()
+      throw err // Re-throwing promise rejection, this is not New Relic's error.
+    }
+  )
 
   function __NR_continueContext() {
     self.setSegment(nextContext.getSegment())
   }
 }
+/* eslint-disable camelcase */

package/lib/instrumentation/redis.js

@@ -19,7 +19,7 @@ module.exports = function initialize(agent, redis, moduleName, shim) {
     shim.recordOperation(
       proto,
       'internal_send_command',
-      function wrapInternalSendCommand(shim, internal_send_command, name, args) {
+      function wrapInternalSendCommand(shim, _, __, args) {
         var commandObject = args[0]
         var keys = commandObject.args
         var parameters = getInstanceParameters(this)
@@ -41,11 +41,15 @@ module.exports = function initialize(agent, redis, moduleName, shim) {
               shim.bindCallbackSegment(commandObject, 'callback', segment)
             } else {
               var self = this
-              commandObject.callback = shim.bindSegment(function __NR_redisCallback(err) {
-                if (err && self.emit instanceof Function) {
-                  self.emit('error', err)
-                }
-              }, segment, true)
+              commandObject.callback = shim.bindSegment(
+                function NRCallback(err) {
+                  if (err && self.emit instanceof Function) {
+                    self.emit('error', err)
+                  }
+                },
+                segment,
+                true
+              )
             }
           }
         }
@@ -53,36 +57,32 @@ module.exports = function initialize(agent, redis, moduleName, shim) {
     )
   } else {
     // For redis versions <2.6.1
-    shim.recordOperation(
-      proto,
-      'send_command',
-      function wrapSendCommand(shim, send_command, name, args) {
-        var keys = args[1]
-        var parameters = getInstanceParameters(this)
+    shim.recordOperation(proto, 'send_command', function wrapSendCommand(shim, _, __, args) {
+      var keys = args[1]
+      var parameters = getInstanceParameters(this)
 
-        if (keys && !shim.isFunction(keys)) {
-          try {
-            parameters.key = stringify(keys[0])
-          } catch (err) {
-            shim.logger.debug(err, 'Failed to stringify redis key for send command')
-            parameters.key = '<unknown>'
-          }
+      if (keys && !shim.isFunction(keys)) {
+        try {
+          parameters.key = stringify(keys[0])
+        } catch (err) {
+          shim.logger.debug(err, 'Failed to stringify redis key for send command')
+          parameters.key = '<unknown>'
         }
+      }
 
-        return {
-          name: args[0] || 'other',
-          parameters: parameters,
-          callback: function bindCallback(shim, _f, _n, segment) {
-            var last = args[args.length - 1]
-            if (shim.isFunction(last)) {
-              shim.bindCallbackSegment(args, shim.LAST, segment)
-            } else if (shim.isArray(last) && shim.isFunction(last[last.length - 1])) {
-              shim.bindCallbackSegment(last, shim.LAST, segment)
-            }
+      return {
+        name: args[0] || 'other',
+        parameters: parameters,
+        callback: function bindCallback(shim, _f, _n, segment) {
+          var last = args[args.length - 1]
+          if (shim.isFunction(last)) {
+            shim.bindCallbackSegment(args, shim.LAST, segment)
+          } else if (shim.isArray(last) && shim.isFunction(last[last.length - 1])) {
+            shim.bindCallbackSegment(last, shim.LAST, segment)
           }
         }
       }
-    )
+    })
   }
 
   function getInstanceParameters(client) {

package/lib/instrumentation/when.js

@@ -8,7 +8,15 @@
 var promInit = require('./promise')
 
 var STATIC_PROMISE_METHODS = [
-  'reject', 'resolve', 'all', 'any', 'some', 'map', 'reduce', 'filter', 'reduceRight'
+  'reject',
+  'resolve',
+  'all',
+  'any',
+  'some',
+  'map',
+  'reduce',
+  'filter',
+  'reduceRight'
 ]
 
 var WHEN_SPEC = {

package/lib/instrumentations.js

@@ -10,29 +10,29 @@ var MODULE_TYPE = require('./shim/constants').MODULE_TYPE
 // Return a new copy of this array every time we're called
 module.exports = function instrumentations() {
   return {
-    'aws-sdk': {module: '@newrelic/aws-sdk'},
-    'amqplib': {type: MODULE_TYPE.MESSAGE},
-    'cassandra-driver': {type: MODULE_TYPE.DATASTORE},
-    'connect': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'bluebird': {type: MODULE_TYPE.PROMISE},
-    'director': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'express': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'fastify': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'generic-pool': {type: MODULE_TYPE.GENERIC},
-    '@hapi/hapi': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'hapi': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'ioredis': {type: MODULE_TYPE.DATASTORE},
-    'koa': {module: '@newrelic/koa'},
-    'memcached': {type: MODULE_TYPE.DATASTORE},
-    'mongodb': {type: MODULE_TYPE.DATASTORE},
-    'mysql': {type: MODULE_TYPE.DATASTORE},
-    'pg': {type: MODULE_TYPE.DATASTORE},
-    'q': {type: null},
-    'redis': {type: MODULE_TYPE.DATASTORE},
-    'restify': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'superagent': {module: '@newrelic/superagent'},
-    'oracle': {type: null},
-    'vision': {type: MODULE_TYPE.WEB_FRAMEWORK},
-    'when': {type: null}
+    'aws-sdk': { module: '@newrelic/aws-sdk' },
+    'amqplib': { type: MODULE_TYPE.MESSAGE },
+    'cassandra-driver': { type: MODULE_TYPE.DATASTORE },
+    'connect': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'bluebird': { type: MODULE_TYPE.PROMISE },
+    'director': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'express': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'fastify': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'generic-pool': { type: MODULE_TYPE.GENERIC },
+    '@hapi/hapi': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'hapi': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'ioredis': { type: MODULE_TYPE.DATASTORE },
+    'koa': { module: '@newrelic/koa' },
+    'memcached': { type: MODULE_TYPE.DATASTORE },
+    'mongodb': { type: MODULE_TYPE.DATASTORE },
+    'mysql': { type: MODULE_TYPE.DATASTORE },
+    'pg': { type: MODULE_TYPE.DATASTORE },
+    'q': { type: null },
+    'redis': { type: MODULE_TYPE.DATASTORE },
+    'restify': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'superagent': { module: '@newrelic/superagent' },
+    'oracle': { type: null },
+    'vision': { type: MODULE_TYPE.WEB_FRAMEWORK },
+    'when': { type: null }
   }
 }

package/lib/logger.js

@@ -43,7 +43,7 @@ if (config) {
         break
 
       default:
-        stream = fs.createWriteStream(config.logging.filepath, {flags: 'a+'})
+        stream = fs.createWriteStream(config.logging.filepath, { flags: 'a+' })
         stream.on('error', function logStreamOnError(err) {
           /* eslint-disable no-console */
           // Since our normal logging didn't work, dump this to stderr.

package/lib/metrics/index.js

@@ -9,7 +9,6 @@ const Stats = require('../stats')
 const ApdexStats = require('../stats/apdex.js')
 const NAMES = require('./names')
 
-
 /*
  *
  * CONSTANTS
@@ -48,10 +47,14 @@ const FROM_MILLIS = 1e-3
  */
 function Metrics(apdexT, mapper, normalizer) {
   if (apdexT == null || apdexT === '') {
-    throw new Error("metrics must be created with apdexT")
+    throw new Error('metrics must be created with apdexT')
+  }
+  if (!mapper) {
+    throw new Error('metrics must be created with a mapper')
+  }
+  if (!normalizer) {
+    throw new Error('metrics must be created with a name normalizer')
   }
-  if (!mapper) throw new Error("metrics must be created with a mapper")
-  if (!normalizer) throw new Error("metrics must be created with a name normalizer")
 
   this.empty = true
   this.started = Date.now()
@@ -134,7 +137,9 @@ Metrics.prototype.getOrCreateMetric = function getOrCreateMetric(name, scope) {
 Metrics.prototype.getOrCreateApdexMetric = getOrCreateApdexMetric
 
 function getOrCreateApdexMetric(name, scope, overrideApdex) {
-  if (!name) throw new Error('Metrics must be named')
+  if (!name) {
+    throw new Error('Metrics must be named')
+  }
 
   var resolved = this._resolve(scope)
 
@@ -143,8 +148,8 @@ function getOrCreateApdexMetric(name, scope, overrideApdex) {
 
     // Only use the given override to create the metric if this is not the
     // global apdex AND we have a valid value.
-    var apdexT = name !== NAMES.APDEX && overrideApdex > 0
-      ? (overrideApdex * FROM_MILLIS) : this.apdexT
+    var apdexT =
+      name !== NAMES.APDEX && overrideApdex > 0 ? overrideApdex * FROM_MILLIS : this.apdexT
     resolved[name] = new ApdexStats(apdexT)
   }
   return resolved[name]
@@ -162,7 +167,9 @@ function getOrCreateApdexMetric(name, scope, overrideApdex) {
  *                  undefined.
  */
 Metrics.prototype.getMetric = function getMetric(name, scope) {
-  if (!name) throw new Error('Metrics must be named')
+  if (!name) {
+    throw new Error('Metrics must be named')
+  }
 
   return this._resolve(scope)[name]
 }
@@ -201,7 +208,8 @@ Metrics.prototype.merge = function merge(other, adjustStartTime) {
 
   // Loop through all scopes and merge them. Since we know `.scoped` has a `null`
   // prototype we don't need to worry about own property checks.
-  for (var scope in other.scoped) { // eslint-disable-line guard-for-in
+  // eslint-disable-next-line guard-for-in
+  for (var scope in other.scoped) {
     _merge(this._resolve(scope), other.scoped[scope])
   }
 }
@@ -243,10 +251,14 @@ Metrics.prototype._resolve = function _resolve(scope) {
  * @param {string} name The string to look up.
  */
 Metrics.prototype._getUnscopedData = function _getUnscopedData(name) {
-  if (!this.unscoped[name]) return
+  if (!this.unscoped[name]) {
+    return
+  }
 
   var normalized = this.normalizer.normalize(name)
-  if (normalized.ignore || !normalized.value) return
+  if (normalized.ignore || !normalized.value) {
+    return
+  }
 
   return [this.mapper.map(normalized.value), this.unscoped[name]]
 }
@@ -258,10 +270,14 @@ Metrics.prototype._getUnscopedData = function _getUnscopedData(name) {
  * @param {string} name The string to look up.
  */
 Metrics.prototype._getScopedData = function _getScopedData(name, scope) {
-  if (!this.scoped[scope][name]) return
+  if (!this.scoped[scope][name]) {
+    return
+  }
 
   var normalized = this.normalizer.normalize(name)
-  if (normalized.ignore || !normalized.value) return
+  if (normalized.ignore || !normalized.value) {
+    return
+  }
 
   return [this.mapper.map(normalized.value, scope), this.scoped[scope][name]]
 }
@@ -273,10 +289,12 @@ Metrics.prototype._getScopedData = function _getScopedData(name, scope) {
 Metrics.prototype._toUnscopedData = function _toUnscopedData() {
   var metricData = []
 
-  Object.keys(this.unscoped).forEach(function cb_forEach(name) {
+  Object.keys(this.unscoped).forEach((name) => {
     var data = this._getUnscopedData(name)
-    if (data) metricData.push(data)
-  }, this)
+    if (data) {
+      metricData.push(data)
+    }
+  })
 
   return metricData
 }
@@ -291,7 +309,9 @@ Metrics.prototype._toScopedData = function _toScopedData() {
   Object.keys(this.scoped).forEach(function forEachScope(scope) {
     Object.keys(this.scoped[scope]).forEach(function forEachMetric(name) {
       var data = this._getScopedData(name, scope)
-      if (data) metricData.push(data)
+      if (data) {
+        metricData.push(data)
+      }
     }, this)
   }, this)
 

package/lib/metrics/mapper.js

@@ -5,8 +5,7 @@
 
 'use strict'
 
-var logger = require('../logger').child({component: 'mapper'})
-
+var logger = require('../logger').child({ component: 'mapper' })
 
 /**
  * To tighten up the metrics JSON, the collector will maintain a list of
@@ -38,7 +37,7 @@ function MetricMapper(raw) {
  */
 MetricMapper.prototype.load = function load(raw) {
   if (!(raw && raw.length)) {
-    logger.debug("No new metric mappings from server.")
+    logger.debug('No new metric mappings from server.')
     return
   }
 
@@ -49,7 +48,6 @@ MetricMapper.prototype.load = function load(raw) {
     var id = raw[i][1]
     var resolved
 
-
     if (scope) {
       if (!this.scoped[scope]) {
         this.scoped[scope] = Object.create(null)
@@ -59,11 +57,13 @@ MetricMapper.prototype.load = function load(raw) {
       resolved = this.unscoped
     }
 
-    if (!resolved[name]) this.length++
+    if (!resolved[name]) {
+      this.length++
+    }
     resolved[name] = id
-    logger.trace("Metric spec %s has been mapped to ID %s.", spec, id)
+    logger.trace('Metric spec %s has been mapped to ID %s.', spec, id)
   }
-  logger.debug("Parsed %d metric ids (%d total).", raw.length, this.length)
+  logger.debug('Parsed %d metric ids (%d total).', raw.length, this.length)
 }
 
 /**
@@ -78,14 +78,14 @@ MetricMapper.prototype.map = function map(name, scope) {
     if (this.scoped[scope] && this.scoped[scope][name]) {
       return this.scoped[scope][name]
     }
-    return {name: name, scope: scope}
+    return { name: name, scope: scope }
   }
 
   if (this.unscoped[name]) {
     return this.unscoped[name]
   }
 
-  return {name: name}
+  return { name: name }
 }
 
 module.exports = MetricMapper

package/lib/metrics/metric-aggregator.js

@@ -5,7 +5,7 @@
 
 'use strict'
 
-const logger = require('../logger').child({component: 'metric-aggregator'})
+const logger = require('../logger').child({ component: 'metric-aggregator' })
 const Aggregator = require('../aggregators/base-aggregator')
 const Metrics = require('../metrics')
 
@@ -43,12 +43,7 @@ class MetricAggregator extends Aggregator {
     const beginSeconds = this._metrics.started * FROM_MILLIS
     const endSeconds = Date.now() * FROM_MILLIS
 
-    const payload = [
-      this.runId,
-      beginSeconds,
-      endSeconds,
-      this._metrics.toJSON()
-    ]
+    const payload = [this.runId, beginSeconds, endSeconds, this._metrics.toJSON()]
 
     return payload
   }
@@ -164,15 +159,15 @@ function _isValidOrThrow(opts) {
   }
 
   if (opts.apdexT == null || opts.apdexT === '') {
-    throw new Error("Metric aggregator must be created with apdexT")
+    throw new Error('Metric aggregator must be created with apdexT')
   }
 
   if (!opts.mapper) {
-    throw new Error("Metric aggregator must be created with a mapper")
+    throw new Error('Metric aggregator must be created with a mapper')
   }
 
   if (!opts.normalizer) {
-    throw new Error("Metric aggregator must be created with a name normalizer")
+    throw new Error('Metric aggregator must be created with a name normalizer')
   }
 }
 

package/lib/metrics/names.js

@@ -143,7 +143,6 @@ const CASSANDRA = {
   STATEMENT: DB.STATEMENT + '/Cassandra/',
   INSTANCE: DB.INSTANCE + '/Cassandra/',
   ALL: DB.PREFIX + 'Cassandra/' + ALL
-
 }
 
 const ORACLE = {
@@ -165,7 +164,7 @@ const RESTIFY = {
 
 const HAPI = {
   PREFIX: 'Hapi/',
-  MIDDLEWARE: MIDDLEWARE.PREFIX + 'Hapi/',
+  MIDDLEWARE: MIDDLEWARE.PREFIX + 'Hapi/'
 }
 
 const UTILIZATION = {
@@ -177,7 +176,6 @@ const UTILIZATION = {
   BOOT_ID_ERROR: SUPPORTABILITY.UTILIZATION + '/boot_id/error'
 }
 
-
 const CUSTOM_EVENTS = {
   PREFIX: SUPPORTABILITY.EVENTS + '/Customer/',
   DROPPED: SUPPORTABILITY.EVENTS + '/Customer/Dropped',
@@ -232,10 +230,13 @@ const DISTRIBUTED_TRACE = {
   TRANSPORT: 'TransportDuration'
 }
 
+const SPAN_EVENT_PREFIX = 'SpanEvent/'
+
 const SPAN_EVENTS = {
-  SEEN: SUPPORTABILITY.PREFIX + 'SpanEvent/TotalEventsSeen',
-  SENT: SUPPORTABILITY.PREFIX + 'SpanEvent/TotalEventsSent',
-  DROPPED: SUPPORTABILITY.PREFIX + 'SpanEvent/Discarded'
+  SEEN: SUPPORTABILITY.PREFIX + SPAN_EVENT_PREFIX + 'TotalEventsSeen',
+  SENT: SUPPORTABILITY.PREFIX + SPAN_EVENT_PREFIX + 'TotalEventsSent',
+  DROPPED: SUPPORTABILITY.PREFIX + SPAN_EVENT_PREFIX + 'Discarded',
+  LIMIT: SUPPORTABILITY.PREFIX + SPAN_EVENT_PREFIX + 'Limit'
 }
 
 const INFINITE_TRACING = {
@@ -247,7 +248,7 @@ const INFINITE_TRACING = {
   SPAN_RESPONSE_GRPC_STATUS: SUPPORTABILITY.INFINITE_TRACING + '/Span/gRPC/%s',
   QUEUE_CAPACITY: SUPPORTABILITY.INFINITE_TRACING + '/Span/QueueCapacity',
   QUEUE_SIZE: SUPPORTABILITY.INFINITE_TRACING + '/Span/QueueSize',
-  DRAIN_DURATION: SUPPORTABILITY.INFINITE_TRACING + '/Drain/Duration',
+  DRAIN_DURATION: SUPPORTABILITY.INFINITE_TRACING + '/Drain/Duration'
 }
 
 const FEATURES = {

package/lib/metrics/normalizer/rule.js

@@ -5,8 +5,7 @@
 
 'use strict'
 
-var logger = require('../../logger').child({component: 'normalizer_rule'})
-
+var logger = require('../../logger').child({ component: 'normalizer_rule' })
 
 /**
  * JavaScript just has to do things slightly differently.
@@ -26,9 +25,7 @@ var replaceReplacer = function replaceReplacer(input) {
  */
 function NormalizerRule(json) {
   if (!json) {
-    logger.debug(
-      "Received incompletely specified metric normalization rule from collector."
-    )
+    logger.debug('Received incompletely specified metric normalization rule from collector.')
     json = Object.create(null)
   }
 
@@ -41,7 +38,9 @@ function NormalizerRule(json) {
   this.matched = false
 
   var modifiers = 'i'
-  if (this.replaceAll) modifiers += 'g'
+  if (this.replaceAll) {
+    modifiers += 'g'
+  }
 
   // don't allow this to fail
   if (json.match_expression instanceof RegExp) {
@@ -50,7 +49,7 @@ function NormalizerRule(json) {
     try {
       this.pattern = new RegExp(json.match_expression || '^$', modifiers)
     } catch (error) {
-      logger.warn(error, "Problem compiling metric normalization rule pattern.")
+      logger.warn(error, 'Problem compiling metric normalization rule pattern.')
       this.pattern = /^$/
     }
   }

package/lib/metrics/normalizer/tx_segment.js

@@ -5,7 +5,7 @@
 
 'use strict'
 
-var logger = require('../../logger').child({component: 'tx_segment_normalizer'})
+var logger = require('../../logger').child({ component: 'tx_segment_normalizer' })
 
 module.exports = TxSegmentNormalizer
 
@@ -50,13 +50,17 @@ TxSegmentNormalizer.prototype.normalize = function normalize(path) {
     for (var j = 0; j < parts.length; j++) {
       segment = parts[j]
 
-      if (segment === '' && j + 1 === parts.length) break
+      if (segment === '' && j + 1 === parts.length) {
+        break
+      }
 
       if (currentTerm.terms.indexOf(segment) === -1) {
-        if (prev === '*') continue
-        result.push(prev = '*')
+        if (prev === '*') {
+          continue
+        }
+        result.push((prev = '*'))
       } else {
-        result.push(prev = segment)
+        result.push((prev = segment))
       }
     }
     logger.trace('Normalizing %s because of rule: %s', path, currentTerm)
@@ -69,7 +73,7 @@ TxSegmentNormalizer.prototype.normalize = function normalize(path) {
 
   return {
     matched: false, // To match MetricNormalizer
-    ignore: false,  // ^^
+    ignore: false, // ^^
     value: path
   }
 }
@@ -78,11 +82,7 @@ TxSegmentNormalizer.prototype.load = function load(json) {
   if (Array.isArray(json)) {
     this.terms = filterRules(json)
   } else {
-    logger.warn(
-      'transaction_segment_terms was not an array got: %s (%s)',
-      typeof json,
-      json
-    )
+    logger.warn('transaction_segment_terms was not an array got: %s (%s)', typeof json, json)
   }
 }
 
@@ -92,7 +92,9 @@ function filterRules(rules) {
   for (var i = 0, l = rules.length; i < l; ++i) {
     var prefix = rules[i].prefix
 
-    if (!prefix || typeof prefix !== 'string') continue
+    if (!prefix || typeof prefix !== 'string') {
+      continue
+    }
 
     if (prefix[prefix.length - 1] !== '/') {
       prefix = prefix + '/'
@@ -100,7 +102,9 @@ function filterRules(rules) {
     }
 
     var segments = prefix.split('/')
-    if (segments.length !== 3 || !segments[0] || !segments[1] || segments[3]) continue
+    if (segments.length !== 3 || !segments[0] || !segments[1] || segments[3]) {
+      continue
+    }
 
     if (Array.isArray(rules[i].terms)) {
       map[prefix] = rules[i]