newrelic 7.5.2 → 8.3.0

package/lib/metrics/normalizer.js

@@ -7,14 +7,15 @@
 
 var EventEmitter = require('events').EventEmitter
 var util = require('util')
-var logger = require('../logger').child({component: 'metric_normalizer'})
+var logger = require('../logger').child({ component: 'metric_normalizer' })
 var deepEqual = require('../util/deep-equal')
 var NormalizerRule = require('./normalizer/rule')
 var NAMES = require('../metrics/names.js')
 
-
 function url(normalized, path, config) {
-  if (normalized) return NAMES.NORMALIZED + normalized
+  if (normalized) {
+    return NAMES.NORMALIZED + normalized
+  }
 
   if (config.enforce_backstop) {
     return NAMES.NORMALIZED + '/*'
@@ -42,8 +43,12 @@ function plain(normalized, path) {
  *                        backstop.
  */
 function MetricNormalizer(config, type) {
-  if (!config) throw new Error("normalizer must be created with configuration.")
-  if (!type) throw new Error("normalizer must be created with a type.")
+  if (!config) {
+    throw new Error('normalizer must be created with configuration.')
+  }
+  if (!type) {
+    throw new Error('normalizer must be created with a type.')
+  }
 
   EventEmitter.call(this)
 
@@ -82,26 +87,25 @@ util.inherits(MetricNormalizer, EventEmitter)
 MetricNormalizer.prototype.load = function load(json) {
   if (json) {
     this.rules = []
-    logger.debug("Received %s %s normalization rule(s) from the server",
-      json.length, this.type)
+    logger.debug('Received %s %s normalization rule(s) from the server', json.length, this.type)
 
-    json.forEach(function cb_forEach(ruleJSON) {
+    json.forEach((ruleJSON) => {
       // no need to add the same rule twice
       var rule = new NormalizerRule(ruleJSON)
       if (!this.rules.find(deepEqual.bind(null, rule))) {
         this.rules.push(rule)
-        logger.trace("Loaded %s normalization rule: %s", this.type, rule)
+        logger.trace('Loaded %s normalization rule: %s', this.type, rule)
       }
-    }, this)
+    })
 
     /* I (FLN) always forget this, so making a note: JS sort is always
      * IN-PLACE, even though it returns the sorted array.
      */
-    this.rules.sort(function cb_sort(a, b) {
+    this.rules.sort((a, b) => {
       return a.precedence - b.precedence
     })
 
-    logger.debug("Loaded %s %s normalization rule(s).", this.rules.length, this.type)
+    logger.debug('Loaded %s %s normalization rule(s).', this.rules.length, this.type)
   }
 }
 
@@ -114,26 +118,20 @@ MetricNormalizer.prototype.loadFromConfig = function loadFromConfig() {
   var rules = this.config.rules
 
   if (rules && rules.name && rules.name.length > 0) {
-    rules.name.forEach(function cb_forEach(rule) {
+    rules.name.forEach((rule) => {
       if (!rule.pattern) {
-        return logger.error(
-          {rule: rule},
-          "Simple naming rules require a pattern."
-        )
+        return logger.error({ rule: rule }, 'Simple naming rules require a pattern.')
       }
       if (!rule.name) {
-        return logger.error(
-          {rule: rule},
-          "Simple naming rules require a replacement name."
-        )
+        return logger.error({ rule: rule }, 'Simple naming rules require a replacement name.')
       }
 
       var precedence = rule.precedence
       var terminal = rule.terminate_chain
       var json = {
         match_expression: rule.pattern,
-        eval_order: (typeof precedence === 'number') ? precedence : 500,
-        terminate_chain: (typeof terminal === 'boolean') ? terminal : true,
+        eval_order: typeof precedence === 'number' ? precedence : 500,
+        terminate_chain: typeof terminal === 'boolean' ? terminal : true,
         replace_all: rule.replace_all,
         replacement: rule.name,
         ignore: false
@@ -142,22 +140,20 @@ MetricNormalizer.prototype.loadFromConfig = function loadFromConfig() {
       // Find where the rule should be inserted and do so.
       var reverse = this.config.feature_flag.reverse_naming_rules
       var insert = this.rules.findIndex(function findRule(r) {
-        return reverse
-          ? r.precedence >= json.eval_order
-          : r.precedence > json.eval_order
+        return reverse ? r.precedence >= json.eval_order : r.precedence > json.eval_order
       })
       if (insert === -1) {
         this.rules.push(new NormalizerRule(json))
       } else {
         this.rules.splice(insert, 0, new NormalizerRule(json))
       }
-    }, this)
+    })
   }
 
   if (rules && rules.ignore && rules.ignore.length > 0) {
-    rules.ignore.forEach(function cb_forEach(pattern) {
+    rules.ignore.forEach((pattern) => {
       this.addSimple(pattern)
-    }, this)
+    })
   }
 }
 
@@ -174,7 +170,9 @@ MetricNormalizer.prototype.loadFromConfig = function loadFromConfig() {
  * @param {string} [name]  The name to use for the transaction.
  */
 MetricNormalizer.prototype.addSimple = function addSimple(pattern, name) {
-  if (!pattern) return logger.error("Simple naming rules require a pattern.")
+  if (!pattern) {
+    return logger.error('Simple naming rules require a pattern.')
+  }
 
   var json = {
     match_expression: pattern,
@@ -227,13 +225,12 @@ MetricNormalizer.prototype.normalize = function normalize(path) {
       // that would increase memory overhead by creating additional array
       this.emit('appliedRule', rule, normalized, last)
 
-      logger.trace({rule: rule, type: this.type},
-        "Normalized %s to %s.", last, normalized)
+      logger.trace({ rule: rule, type: this.type }, 'Normalized %s to %s.', last, normalized)
       last = normalized
     }
 
     if (rule.isTerminal) {
-      logger.trace({rule: rule}, "Terminating normalization.")
+      logger.trace({ rule: rule }, 'Terminating normalization.')
       break
     }
   }

package/lib/metrics/recorders/custom.js

@@ -13,7 +13,9 @@ function record(segment, scope) {
   var transaction = segment.transaction
   var name = NAMES.CUSTOM + NAMES.ACTION_DELIMITER + segment.name
 
-  if (scope) transaction.measure(name, scope, duration, exclusive)
+  if (scope) {
+    transaction.measure(name, scope, duration, exclusive)
+  }
 
   transaction.measure(name, null, duration, exclusive)
 }

package/lib/metrics/recorders/distributed-trace.js

@@ -20,29 +20,14 @@ function recordDistributedTrace(tx, suffix, duration, exclusive) {
   const suffixes = ['', suffix]
 
   suffixes.forEach(function record(suf) {
-    tx.measure(
-      `${NAMES.DISTRIBUTED_TRACE.DURATION}/${tag}${suf}`,
-      null,
-      duration,
-      exclusive
-    )
+    tx.measure(`${NAMES.DISTRIBUTED_TRACE.DURATION}/${tag}${suf}`, null, duration, exclusive)
 
     if (tx.hasErrors()) {
-      tx.measure(
-        `${NAMES.DISTRIBUTED_TRACE.ERRORS}/${tag}${suf}`,
-        null,
-        duration,
-        exclusive
-      )
+      tx.measure(`${NAMES.DISTRIBUTED_TRACE.ERRORS}/${tag}${suf}`, null, duration, exclusive)
     }
 
     if (distTraceReceived) {
-      tx.measure(
-        `${NAMES.DISTRIBUTED_TRACE.TRANSPORT}/${tag}${suf}`,
-        null,
-        duration,
-        exclusive
-      )
+      tx.measure(`${NAMES.DISTRIBUTED_TRACE.TRANSPORT}/${tag}${suf}`, null, duration, exclusive)
     }
   })
 }

package/lib/metrics/recorders/generic.js

@@ -10,8 +10,9 @@ function record(segment, scope) {
   var exclusive = segment.getExclusiveDurationInMillis()
   var transaction = segment.transaction
 
-
-  if (scope) transaction.measure(segment.name, scope, duration, exclusive)
+  if (scope) {
+    transaction.measure(segment.name, scope, duration, exclusive)
+  }
 
   transaction.measure(segment.name, null, duration, exclusive)
 }

package/lib/metrics/recorders/http.js

@@ -12,7 +12,9 @@ const TO_MILLIS = 1e3
 
 function recordWeb(segment, scope) {
   // in web metrics, scope is required
-  if (!scope) return
+  if (!scope) {
+    return
+  }
 
   var tx = segment.transaction
   // if there was a nested webTransaction use its recorder instead
@@ -41,11 +43,7 @@ function recordWeb(segment, scope) {
   if (config.distributed_tracing.enabled) {
     recordDistributedTrace(tx, 'Web', duration, exclusive)
   } else if (tx.incomingCatId) {
-    tx.measure(
-      NAMES.CLIENT_APPLICATION + '/' + tx.incomingCatId + "/all",
-      null,
-      tx.catResponseTime
-    )
+    tx.measure(NAMES.CLIENT_APPLICATION + '/' + tx.incomingCatId + '/all', null, tx.catResponseTime)
   }
 
   tx._setApdex(NAMES.APDEX + '/' + partial, duration, keyApdexInMillis)

package/lib/metrics/recorders/http_external.js

@@ -7,7 +7,6 @@
 
 var EXTERNAL = require('../../metrics/names').EXTERNAL
 
-
 function recordExternal(host, library) {
   return function externalRecorder(segment, scope) {
     var duration = segment.getDurationInMillis()
@@ -35,8 +34,7 @@ function recordExternal(host, library) {
       // This CAT metric replaces scoped External/{host}/{method}
       if (scope) {
         transaction.measure(
-          EXTERNAL.TRANSACTION + host + '/' + segment.catId +
-            '/' + segment.catTransaction,
+          EXTERNAL.TRANSACTION + host + '/' + segment.catId + '/' + segment.catTransaction,
           scope,
           duration,
           exclusive

package/lib/metrics/recorders/message-transaction.js

@@ -28,18 +28,8 @@ function recordMessageTransaction(segment, scope) {
     )
   }
 
-  tx.measure(
-    NAMES.MESSAGE_TRANSACTION.RESPONSE_TIME + '/all',
-    null,
-    duration,
-    exclusive
-  )
-  tx.measure(
-    NAMES.OTHER_TRANSACTION.RESPONSE_TIME + '/all',
-    null,
-    duration,
-    exclusive
-  )
+  tx.measure(NAMES.MESSAGE_TRANSACTION.RESPONSE_TIME + '/all', null, duration, exclusive)
+  tx.measure(NAMES.OTHER_TRANSACTION.RESPONSE_TIME + '/all', null, duration, exclusive)
   tx.measure(NAMES.OTHER_TRANSACTION.TOTAL_TIME, null, totalTime, exclusive)
 }
 

package/lib/metrics/recorders/other.js

@@ -22,21 +22,11 @@ function recordBackground(segment, scope) {
 
   if (scope) {
     tx.measure(scope, null, duration, exclusive)
-    tx.measure(
-      NAMES.OTHER_TRANSACTION.TOTAL_TIME + '/' + name,
-      null,
-      totalTime,
-      exclusive
-    )
+    tx.measure(NAMES.OTHER_TRANSACTION.TOTAL_TIME + '/' + name, null, totalTime, exclusive)
   }
   // rollup for background total time doesn't have `/all` where the response
   // time version does.
-  tx.measure(
-    NAMES.OTHER_TRANSACTION.RESPONSE_TIME + '/all',
-    null,
-    duration,
-    exclusive
-  )
+  tx.measure(NAMES.OTHER_TRANSACTION.RESPONSE_TIME + '/all', null, duration, exclusive)
   tx.measure(NAMES.OTHER_TRANSACTION.TOTAL_TIME, null, totalTime, exclusive)
 
   if (tx.agent.config.distributed_tracing.enabled) {

package/lib/parse-proc-cpuinfo.js

@@ -5,16 +5,11 @@
 
 'use strict'
 
-var logger = require('./logger.js').child({component: 'proc-cpuinfo'})
+var logger = require('./logger.js').child({ component: 'proc-cpuinfo' })
 module.exports = parseProcCPUInfo
 
 function parseProcCPUInfo(data) {
-  var relevantAttributes = [
-    'processor',
-    'physical id',
-    'cpu cores',
-    'core id'
-  ]
+  var relevantAttributes = ['processor', 'physical id', 'cpu cores', 'core id']
 
   var processorStats = {
     logical: null,
@@ -23,13 +18,11 @@ function parseProcCPUInfo(data) {
   }
 
   // separate the processors
-  var splitData = data.split('\n')
-    .map(function formatAttribute(attr) {
-      return attr.split(':')
-        .map(function eliminateExtraWhitespace(s) {
-          return s.replace(/\\r|\\t| {2,}/g, '').trim()
-        })
+  var splitData = data.split('\n').map(function formatAttribute(attr) {
+    return attr.split(':').map(function eliminateExtraWhitespace(s) {
+      return s.replace(/\\r|\\t| {2,}/g, '').trim()
     })
+  })
 
   var validData = splitData.filter(function checkForValidAttrs(a) {
     return a.length === 2 && relevantAttributes.indexOf(a[0]) !== -1
@@ -102,9 +95,11 @@ function countProcessorStats(processors) {
 
   for (var i = 0; i < processors.length; i++) {
     var processor = processors[i]
-    if (processor['physical id'] &&
-        processor['cpu cores'] &&
-        phys.indexOf(processor['physical id']) === -1) {
+    if (
+      processor['physical id'] &&
+      processor['cpu cores'] &&
+      phys.indexOf(processor['physical id']) === -1
+    ) {
       phys.push(processor['physical id'])
       cores.push(processor['cpu cores'])
     }

package/lib/parse-proc-meminfo.js

@@ -5,14 +5,16 @@
 
 'use strict'
 
-var logger = require('./logger.js').child({component: 'proc-meminfo'})
+var logger = require('./logger.js').child({ component: 'proc-meminfo' })
 
 module.exports = parseProcMeminfo
 
 function parseProcMeminfo(data) {
-  var mem_total = parseInt(data.replace(/MemTotal:\s*(\d*)\skB/, '$1'), 10)
+  var memTotal = parseInt(data.replace(/MemTotal:\s*(\d*)\skB/, '$1'), 10)
 
-  if (mem_total) return mem_total / 1024
+  if (memTotal) {
+    return memTotal / 1024
+  }
 
   logger.debug('Unable to parse memory string:', data)
   return null

package/lib/prioritized-attributes.js

@@ -6,7 +6,7 @@
 'use strict'
 
 const Config = require('./config')
-const logger = require('./logger').child({component: 'attributes'})
+const logger = require('./logger').child({ component: 'attributes' })
 const isValidType = require('./util/attribute-types')
 const byteUtils = require('./util/byte-limit')
 const properties = require('./util/properties')
@@ -24,13 +24,12 @@ class PrioritizedAttributes {
     this.attributes = new Map()
   }
 
-
   isValidLength(str) {
     return typeof str === 'number' || byteUtils.isValidLength(str, 255)
   }
 
   _set(destinations, key, value, truncateExempt, priority) {
-    this.attributes.set(key, {value, destinations, truncateExempt, priority})
+    this.attributes.set(key, { value, destinations, truncateExempt, priority })
   }
 
   get(dest) {
@@ -41,9 +40,10 @@ class PrioritizedAttributes {
         continue
       }
 
-      attrs[key] = typeof attr.value === 'string' && !attr.truncateExempt
-        ? byteUtils.truncate(attr.value, 255)
-        : attr.value
+      attrs[key] =
+        typeof attr.value === 'string' && !attr.truncateExempt
+          ? byteUtils.truncate(attr.value, 255)
+          : attr.value
     }
 
     return attrs
@@ -81,7 +81,7 @@ class PrioritizedAttributes {
     }
 
     if (existingAttribute && priority < existingAttribute.priority) {
-      logger.debug('incoming priority for \'%s\' is lower than existing, not updating.', key)
+      logger.debug("incoming priority for '%s' is lower than existing, not updating.", key)
       logger.trace(
         '%s attribute retained value: %s, ignored value: %s',
         key,
@@ -94,8 +94,8 @@ class PrioritizedAttributes {
     if (!isValidType(value)) {
       logger.debug(
         'Not adding attribute %s with %s value type. This is expected for undefined' +
-        'attributes and only an issue if an attribute is not expected to be undefined' +
-        'or not of the type expected.',
+          'attributes and only an issue if an attribute is not expected to be undefined' +
+          'or not of the type expected.',
         key,
         typeof value
       )
@@ -103,14 +103,10 @@ class PrioritizedAttributes {
     }
 
     if (!this.isValidLength(key)) {
-      logger.warn(
-        'Length limit exceeded for attribute name, not adding: %s',
-        key
-      )
+      logger.warn('Length limit exceeded for attribute name, not adding: %s', key)
       return
     }
 
-
     // Only set the attribute if at least one destination passed
     const validDestinations = this.filter(destinations, key)
     if (!validDestinations) {
@@ -119,8 +115,7 @@ class PrioritizedAttributes {
 
     if (droppableAttributeKey) {
       logger.trace(
-        'dropping existing lower priority attribute %s ' +
-        'to add higher priority attribute %s',
+        'dropping existing lower priority attribute %s ' + 'to add higher priority attribute %s',
         droppableAttributeKey,
         key
       )
@@ -199,7 +194,7 @@ class PrioritizedAttributes {
 }
 
 function makeFilter(scope) {
-  const {attributeFilter} = Config.getInstance()
+  const { attributeFilter } = Config.getInstance()
   if (scope === 'transaction') {
     return (d, k) => attributeFilter.filterTransaction(d, k)
   } else if (scope === 'segment') {

package/lib/priority-queue.js

@@ -64,7 +64,7 @@ PriorityQueue.prototype.getRawEvents = function getRawEvents() {
     var current = head
 
     do {
-      evts.push({value: current.value, priority: current.key})
+      evts.push({ value: current.value, priority: current.key })
       if (current.child) {
         _getRawEvents(current.child, evts)
       }

package/lib/proxy/grpc.js

@@ -17,4 +17,4 @@ class ProxyGrpc {
     this.ServerCredentials = this.library.ServerCredentials
   }
 }
-module.exports = (new ProxyGrpc)
+module.exports = new ProxyGrpc()

package/lib/reservoir.js

@@ -30,7 +30,9 @@ Reservoir.prototype.add = function add(item) {
     // end, swapping the last element (the new one) with a random element in the list,
     // then dropping the last element (the potentially swapped one) in the list.
     var toReplace = Math.floor(Math.random() * (this.seen + 2))
-    if (toReplace < this.limit) this._data[toReplace] = item
+    if (toReplace < this.limit) {
+      this._data[toReplace] = item
+    }
   }
   this.seen++
 }
@@ -40,8 +42,12 @@ Reservoir.prototype.toArray = function toArray() {
 }
 
 Reservoir.prototype.merge = function merge(items) {
-  if (!items || !items.length) return
-  if (items === this._data) return
+  if (!items || !items.length) {
+    return
+  }
+  if (items === this._data) {
+    return
+  }
   for (var i = 0; i < items.length; i++) {
     this.add(items[i])
   }

package/lib/sampler.js

@@ -6,7 +6,7 @@
 'use strict'
 
 var NAMES = require('./metrics/names')
-var logger = require('./logger').child({component: 'sampler'})
+var logger = require('./logger').child({ component: 'sampler' })
 var Timer = require('./timer')
 var os = require('os')
 
@@ -85,7 +85,7 @@ function generateCPUMetricRecorder(agent) {
     lastSampleTime = Date.now()
 
     var userUtil = userTime / totalCpuTime
-    var sysUtil  = sysTime / totalCpuTime
+    var sysUtil = sysTime / totalCpuTime
 
     recordValue(agent, NAMES.CPU.USER_TIME, userTime)
     recordValue(agent, NAMES.CPU.SYSTEM_TIME, sysTime)
@@ -149,7 +149,7 @@ function sampleGc(agent, nativeMetrics) {
   }
 }
 
-var sampler = module.exports = {
+var sampler = (module.exports = {
   state: 'stopped',
   sampleMemory: sampleMemory,
   checkEvents: checkEvents,
@@ -170,12 +170,12 @@ var sampler = module.exports = {
         })
       } catch (err) {
         logger.info(
-          {error: {message: err.message, stack: err.stack}},
+          { error: { message: err.message, stack: err.stack } },
           'Not adding native metric sampler.'
         )
-        agent.metrics.getOrCreateMetric(
-          NAMES.SUPPORTABILITY.DEPENDENCIES + '/NoNativeMetricsModule'
-        ).incrementCallCount()
+        agent.metrics
+          .getOrCreateMetric(NAMES.SUPPORTABILITY.DEPENDENCIES + '/NoNativeMetricsModule')
+          .incrementCallCount()
       }
     }
 
@@ -197,12 +197,11 @@ var sampler = module.exports = {
 
     // Add CPU sampling using the built-in data if available, otherwise pulling
     // from the native module.
-    if (process.cpuUsage) { // introduced in 6.1.0
+    if (process.cpuUsage) {
+      // introduced in 6.1.0
       samplers.push(new Sampler(sampleCpu(agent), SAMPLE_INTERVAL))
     } else if (this.nativeMetrics && this.nativeMetrics.usageEnabled) {
-      samplers.push(
-        new Sampler(sampleCpuNative(agent, this.nativeMetrics), SAMPLE_INTERVAL)
-      )
+      samplers.push(new Sampler(sampleCpuNative(agent, this.nativeMetrics), SAMPLE_INTERVAL))
     } else {
       logger.debug('Not adding CPU metric sampler.')
     }
@@ -225,7 +224,7 @@ var sampler = module.exports = {
       this.nativeMetrics = null
     }
   }
-}
+})
 
 function recordValue(agent, metric, value) {
   var stats = agent.metrics.getOrCreateMetric(metric)
@@ -243,5 +242,5 @@ function divideMetric(metric, divisor) {
   metric.min /= divisor
   metric.max /= divisor
   metric.total /= divisor
-  metric.sumOfSquares /= (divisor * divisor)
+  metric.sumOfSquares /= divisor * divisor
 }

package/lib/serverless/api-gateway.js

@@ -14,9 +14,9 @@ class LambdaProxyWebRequest {
   constructor(event) {
     this.headers = event.headers
     this.url = {
-      'path': event.path,
-      'port': event.headers['X-Forwarded-Port'],
-      'requestParameters': event.queryStringParameters
+      path: event.path,
+      port: event.headers['X-Forwarded-Port'],
+      requestParameters: event.queryStringParameters
     }
     this.method = event.httpMethod
     this.transportType = event.headers['X-Forwarded-Proto']