newrelic 7.5.2 → 8.3.0

package/lib/config/index.js

@@ -10,14 +10,13 @@ const CollectorResponse = require('../collector/response')
 const copy = require('../util/copy')
 const defaultConfig = require('./default').config
 const EventEmitter = require('events').EventEmitter
-const feature_flag = require('../feature_flags')
+const featureFlag = require('../feature_flags')
 const flatten = require('../util/flatten')
 const fs = require('../util/unwrapped-core').fs
 const hashes = require('../util/hashes')
 const os = require('os')
 const parseKey = require('../collector/key-parser').parseKey
 const path = require('path')
-const psemver = require('../util/process-version')
 const stringify = require('json-stringify-safe')
 const util = require('util')
 const MergeServerConfig = require('./merge-server-config')
@@ -30,38 +29,32 @@ const mergeServerConfig = new MergeServerConfig()
 const AZURE_APP_NAME = 'APP_POOL_ID'
 const DEFAULT_MAX_PAYLOAD_SIZE_IN_BYTES = 1000000
 const DEFAULT_CONFIG_PATH = require.resolve('./default')
-const SPAN_EVENT_LIMIT = 1000
 const BASE_CONFIG_PATH = require.resolve('../../newrelic')
-const HAS_ARBITRARY_KEYS = new Set([
-  'ignore_messages',
-  'expected_messages',
-  'labels'
-])
+const HAS_ARBITRARY_KEYS = new Set(['ignore_messages', 'expected_messages', 'labels'])
 
 const LASP_MAP = require('./lasp').LASP_MAP
 const ENV = require('./env')
 const HSM = require('./hsm')
+const REMOVE_BEFORE_SEND = new Set(['attributeFilter'])
 
 const exists = fs.existsSync || path.existsSync
 let logger = null // Lazy-loaded in `initialize`.
 let _configInstance = null
 
-const getConfigFileNames = () => [
-  process.env.NEW_RELIC_CONFIG_FILENAME,
-  'newrelic.js',
-  'newrelic.cjs'
-].filter(Boolean)
-
-const getConfigFileLocations = () => [
-  process.env.NEW_RELIC_HOME,
-  process.cwd(),
-  process.env.HOME,
-  path.join(__dirname, '../../../..'), // above node_modules
-  // the REPL has no main module
-  process.mainModule && process.mainModule.filename
-    ? path.dirname(process.mainModule.filename)
-    : undefined
-].filter(Boolean)
+const getConfigFileNames = () =>
+  [process.env.NEW_RELIC_CONFIG_FILENAME, 'newrelic.js', 'newrelic.cjs'].filter(Boolean)
+
+const getConfigFileLocations = () =>
+  [
+    process.env.NEW_RELIC_HOME,
+    process.cwd(),
+    process.env.HOME,
+    path.join(__dirname, '../../../..'), // above node_modules
+    // the REPL has no main module
+    process.mainModule && process.mainModule.filename
+      ? path.dirname(process.mainModule.filename)
+      : undefined
+  ].filter(Boolean)
 
 function isTruthular(setting) {
   if (setting == null) {
@@ -94,7 +87,7 @@ function fromObjectList(setting) {
 
 function _findConfigFile() {
   const configFileCandidates = getConfigFileLocations().reduce((files, configPath) => {
-    const configFiles = getConfigFileNames().map(filename => 
+    const configFiles = getConfigFileNames().map((filename) =>
       path.join(path.resolve(configPath), filename)
     )
 
@@ -118,7 +111,7 @@ function Config(config) {
 
   // feature flags are mostly private settings for gating unreleased features
   // flags are set in the feature_flags.js file
-  this.feature_flag = copy.shallow(feature_flag.prerelease)
+  this.feature_flag = copy.shallow(featureFlag.prerelease)
 
   // set by environment
   this.newrelic_home = null
@@ -161,9 +154,9 @@ function Config(config) {
   this.browser_monitoring.loader_version = ''
 
   // Settings to play nice with DLPs (see NODE-1044).
-  this.compressed_content_encoding = 'deflate'  // Deflate or gzip
-  this.simple_compression = false               // Disables subcomponent compression
-  this.put_for_data_send = false                // Changes http verb for harvest
+  this.compressed_content_encoding = 'deflate' // Deflate or gzip
+  this.simple_compression = false // Disables subcomponent compression
+  this.put_for_data_send = false // Changes http verb for harvest
 
   // 3. override defaults with values from the loaded / passed configuration
   this._fromPassed(config)
@@ -189,7 +182,7 @@ function Config(config) {
         analytic_event_data: this.transaction_events.max_samples_stored,
         custom_event_data: this.custom_insights_events.max_samples_stored,
         error_event_data: this.error_collector.max_event_samples_stored,
-        span_event_data: SPAN_EVENT_LIMIT
+        span_event_data: this.span_events.max_samples_stored
       }
     }
   }
@@ -202,9 +195,9 @@ function Config(config) {
     if (this.security_policies_token) {
       throw new Error(
         'Security Policies and High Security Mode cannot both be present ' +
-        'in the agent configuration. If Security Policies have been set ' +
-        'for your account, please ensure the security_policies_token is ' +
-        'set but high_security is disabled (default).'
+          'in the agent configuration. If Security Policies have been set ' +
+          'for your account, please ensure the security_policies_token is ' +
+          'set but high_security is disabled (default).'
       )
     }
     this._applyHighSecurity()
@@ -244,7 +237,9 @@ Config.prototype.onConnect = function onConnect(json, recursion) {
     this.emit('agent_enabled', false)
     return
   }
-  if (Object.keys(json).length === 0) return
+  if (Object.keys(json).length === 0) {
+    return
+  }
 
   Object.keys(json).forEach(function updateProp(key) {
     this._fromServer(json, key)
@@ -300,8 +295,7 @@ Config.prototype._fromServer = function _fromServer(params, key) {
     case 'encoding_key':
       this._alwaysUpdateIfChanged(params, key)
       if (this.cross_process_id && this.encoding_key) {
-        this.obfuscatedId =
-          hashes.obfuscateNameUsingKey(this.cross_process_id, this.encoding_key)
+        this.obfuscatedId = hashes.obfuscateNameUsingKey(this.cross_process_id, this.encoding_key)
       }
       break
 
@@ -322,12 +316,7 @@ Config.prototype._fromServer = function _fromServer(params, key) {
 
     case 'collect_error_events':
       if (params.collect_error_events === false) {
-        this._updateNestedIfChanged(
-          params,
-          this.error_collector,
-          key,
-          'capture_events'
-        )
+        this._updateNestedIfChanged(params, this.error_collector, key, 'capture_events')
       }
       break
 
@@ -356,10 +345,7 @@ Config.prototype._fromServer = function _fromServer(params, key) {
         this.emit(key, null)
         break
       }
-      logger.info(
-        'Valid event_harvest_config received. Updating harvest cycles.',
-        val
-      )
+      logger.info('Valid event_harvest_config received. Updating harvest cycles.', val)
       const limits = Object.keys(val.harvest_limits).reduce((acc, k) => {
         const v = val.harvest_limits[k]
         if (harvestConfigValidator.isValidHarvestValue(v)) {
@@ -407,12 +393,7 @@ Config.prototype._fromServer = function _fromServer(params, key) {
     // Browser Monitoring
     //
     case 'browser_monitoring.loader':
-      this._updateNestedIfChangedRaw(
-        params,
-        this.browser_monitoring,
-        key,
-        'loader'
-      )
+      this._updateNestedIfChangedRaw(params, this.browser_monitoring, key, 'loader')
       break
 
     // these are used by browser_monitoring
@@ -423,24 +404,14 @@ Config.prototype._fromServer = function _fromServer(params, key) {
     case 'error_beacon':
     case 'browser_key':
     case 'js_agent_loader':
-      this._updateNestedIfChangedRaw(
-        params,
-        this.browser_monitoring,
-        key,
-        key
-      )
+      this._updateNestedIfChangedRaw(params, this.browser_monitoring, key, key)
       break
 
     //
     // Cross Application Tracer
     //
     case 'cross_application_tracer.enabled':
-      this._updateNestedIfChanged(
-        params,
-        this.cross_application_tracer,
-        key,
-        'enabled'
-      )
+      this._updateNestedIfChanged(params, this.cross_application_tracer, key, 'enabled')
       break
     //
     // Error Collector
@@ -531,12 +502,7 @@ Config.prototype._fromServer = function _fromServer(params, key) {
     // Transaction Events
     //
     case 'transaction_events.enabled':
-      this._updateNestedIfChanged(
-        params,
-        this.transaction_events,
-        key,
-        'enabled'
-      )
+      this._updateNestedIfChanged(params, this.transaction_events, key, 'enabled')
       break
 
     //
@@ -577,6 +543,13 @@ Config.prototype._fromServer = function _fromServer(params, key) {
       this.logUnsupported(params, key)
       break
 
+    // DT span event harvest config limits
+    case 'span_event_harvest_config':
+      this.span_event_harvest_config = {
+        ...params[key]
+      }
+      break
+
     // These settings are not allowed from the server.
     case 'attributes.enabled':
     case 'attributes.exclude':
@@ -615,7 +588,9 @@ Config.prototype._alwaysUpdateIfChanged = function _alwaysUpdateIfChanged(json,
   if (value != null && this[key] !== value) {
     if (Array.isArray(value) && Array.isArray(this[key])) {
       value.forEach(function pushIfNew(element) {
-        if (this[key].indexOf(element) === -1) this[key].push(element)
+        if (this[key].indexOf(element) === -1) {
+          this[key].push(element)
+        }
       }, this)
     } else {
       this[key] = value
@@ -664,10 +639,11 @@ function _validateThenUpdateStatusCodes(remote, local, remoteKey, localKey) {
 
   let valid = true
   valueToTest.forEach(function validateArray(thingToTest) {
-    if (!('string' === (typeof thingToTest) || 'number' === (typeof thingToTest))) {
+    if (!('string' === typeof thingToTest || 'number' === typeof thingToTest)) {
       logger.warn(
         'Saw SSC (ignore|expect)_status_code that is not a number or string,' +
-        'will not merge: %s', thingToTest
+          'will not merge: %s',
+        thingToTest
       )
       valid = false
     }
@@ -708,7 +684,7 @@ function _validateThenUpdateErrorClasses(remote, local, remoteKey, localKey) {
   let valid = true
   Object.keys(valueToTest).forEach(function validateArray(key) {
     let thingToTest = valueToTest[key]
-    if ('string' !== (typeof thingToTest)) {
+    if ('string' !== typeof thingToTest) {
       logger.warn(
         'Saw SSC (ignore|expect)_class that is not a string, will not merge: %s',
         thingToTest
@@ -742,10 +718,7 @@ Config.prototype._validateThenUpdateErrorMessages = _validateThenUpdateErrorMess
 function _validateThenUpdateErrorMessages(remote, local, remoteKey, localKey) {
   let valueToTest = remote[remoteKey]
   if (Array.isArray(valueToTest)) {
-    logger.warn(
-      'Saw SSC (ignore|expect)_message that is an Array, will not merge: %s',
-      valueToTest
-    )
+    logger.warn('Saw SSC (ignore|expect)_message that is an Array, will not merge: %s', valueToTest)
     return
   }
 
@@ -771,10 +744,7 @@ function _validateThenUpdateErrorMessages(remote, local, remoteKey, localKey) {
   Object.keys(valueToTest).forEach(function validateArray(key) {
     let arrayToTest = valueToTest[key]
     if (!Array.isArray(arrayToTest)) {
-      logger.warn(
-        'Saw SSC message array that is not an array, will not merge: %s',
-        arrayToTest
-      )
+      logger.warn('Saw SSC message array that is not an array, will not merge: %s', arrayToTest)
       valid = false
     }
   })
@@ -824,7 +794,9 @@ function _updateNestedIfChangedRaw(remote, local, remoteKey, localKey) {
  */
 Config.prototype._emitIfSet = function _emitIfSet(json, key) {
   var value = json[key]
-  if (value != null) this.emit(key, value)
+  if (value != null) {
+    this.emit(key, value)
+  }
 }
 
 /**
@@ -839,7 +811,7 @@ Config.prototype.logDisabled = function logDisabled(json, key) {
   if (value != null) {
     logger.debug(
       'Server-side configuration of %s is currently disabled by local configuration. ' +
-      '(Server sent value of %s.)',
+        '(Server sent value of %s.)',
       key,
       value
     )
@@ -858,7 +830,7 @@ Config.prototype.logUnsupported = function logUnsupported(json, key) {
   if (value !== null && value !== undefined) {
     logger.debug(
       'Server-side configuration of %s is currently not supported by the ' +
-      'Node.js agent. (Server sent value of %s.)',
+        'Node.js agent. (Server sent value of %s.)',
       key,
       value
     )
@@ -874,21 +846,14 @@ Config.prototype.logUnsupported = function logUnsupported(json, key) {
  */
 Config.prototype.logUnknown = function logUnknown(json, key) {
   var value = json[key]
-  logger.debug(
-    'New Relic sent unknown configuration parameter %s with value %s.',
-    key,
-    value
-  )
+  logger.debug('New Relic sent unknown configuration parameter %s with value %s.', key, value)
 }
 
 /**
  * Return the availability of async_hook for use by the agent.
  */
 Config.prototype.checkAsyncHookStatus = function checkAsyncHookStatus() {
-  return (
-    this.feature_flag.await_support &&
-    (psemver.satisfies('>=8') || psemver.prerelease())
-  )
+  return this.feature_flag.await_support
 }
 
 /**
@@ -949,7 +914,9 @@ Config.prototype.getIPAddresses = function getIPAddresses() {
   var interfaces = os.networkInterfaces()
 
   for (var interfaceKey in interfaces) {
-    if (interfaceKey.match(/^lo/)) continue
+    if (interfaceKey.match(/^lo/)) {
+      continue
+    }
 
     var interfaceDescriptions = interfaces[interfaceKey]
     for (var i = 0; i < interfaceDescriptions.length; i++) {
@@ -1013,12 +980,18 @@ Config.prototype.applications = function applications() {
  * @param {object} internal Whichever chunk of the config being overridden.
  */
 Config.prototype._fromPassed = function _fromPassed(external, internal, arbitrary) {
-  if (!external) return
-  if (!internal) internal = this
+  if (!external) {
+    return
+  }
+  if (!internal) {
+    internal = this
+  }
 
   Object.keys(external).forEach(function overwrite(key) {
     // if it's not in the defaults, it doesn't exist
-    if (!arbitrary && internal[key] === undefined) return
+    if (!arbitrary && internal[key] === undefined) {
+      return
+    }
 
     if (key === 'ssl' && !isTruthular(external.ssl)) {
       logger.warn('SSL config key can no longer be disabled, not updating.')
@@ -1052,10 +1025,16 @@ Config.prototype._fromPassed = function _fromPassed(external, internal, arbitrar
  */
 Config.prototype._fromSpecial = function _fromSpecial() {
   var name = this.app_name
-  if (name === null || name === undefined || name === '' ||
-      (Array.isArray(name) && name.length === 0)) {
+  if (
+    name === null ||
+    name === undefined ||
+    name === '' ||
+    (Array.isArray(name) && name.length === 0)
+  ) {
     var azureName = process.env[AZURE_APP_NAME]
-    if (azureName) this.app_name = azureName.split(',')
+    if (azureName) {
+      this.app_name = azureName.split(',')
+    }
   }
 }
 
@@ -1064,7 +1043,7 @@ Config.prototype._fromSpecial = function _fromSpecial() {
  * (of the form NEW_RELIC_FEATURE_FLAG_<feature flag name in upper case>).
  */
 Config.prototype._featureFlagsFromEnv = function _featureFlagsFromEnv() {
-  const flags = Object.keys(feature_flag.prerelease).concat(feature_flag.released)
+  const flags = Object.keys(featureFlag.prerelease).concat(featureFlag.released)
   const config = this
   flags.forEach(function checkFlag(flag) {
     const envVal = process.env['NEW_RELIC_FEATURE_FLAG_' + flag.toUpperCase()]
@@ -1085,8 +1064,12 @@ Config.prototype._featureFlagsFromEnv = function _featureFlagsFromEnv() {
  *                        never need to set this yourself.
  */
 Config.prototype._fromEnvironment = function _fromEnvironment(metadata, data) {
-  if (!metadata) metadata = ENV.ENV_MAPPING
-  if (!data) data = this
+  if (!metadata) {
+    metadata = ENV.ENV_MAPPING
+  }
+  if (!data) {
+    data = this
+  }
 
   Object.keys(metadata).forEach(function applyEnvDefault(value) {
     // if it's not in the config, it doesn't exist
@@ -1123,7 +1106,9 @@ Config.prototype._fromEnvironment = function _fromEnvironment(metadata, data) {
       }
     } else {
       // don't crash if the mapping has config keys the current config doesn't.
-      if (!data[value]) data[value] = Object.create(null)
+      if (!data[value]) {
+        data[value] = Object.create(null)
+      }
       this._fromEnvironment(node, data[value])
     }
   }, this)
@@ -1141,7 +1126,7 @@ Config.prototype._loggingManuallySet = function _loggingManuallySet(inputConfig)
   const inputEnabled = inputConfig && inputConfig.logging && inputConfig.logging.enabled
   const envEnabled = process.env.NEW_RELIC_LOG_ENABLED
 
-  return (inputEnabled !== undefined) || (envEnabled !== undefined)
+  return inputEnabled !== undefined || envEnabled !== undefined
 }
 
 /**
@@ -1152,16 +1137,16 @@ Config.prototype._loggingManuallySet = function _loggingManuallySet(inputConfig)
  *
  * @returns {boolean}
  */
-Config.prototype._nativeMetricsManuallySet =
-  function _nativeMetricsManuallySet(inputConfig) {
-    const inputEnabled = inputConfig
-      && inputConfig.plugins
-      && inputConfig.plugins.native_metrics
-      && inputConfig.plugins.native_metrics.enabled
-    const envEnabled = process.env.NEW_RELIC_NATIVE_METRICS_ENABLED
-
-    return (inputEnabled !== undefined) || (envEnabled !== undefined)
-  }
+Config.prototype._nativeMetricsManuallySet = function _nativeMetricsManuallySet(inputConfig) {
+  const inputEnabled =
+    inputConfig &&
+    inputConfig.plugins &&
+    inputConfig.plugins.native_metrics &&
+    inputConfig.plugins.native_metrics.enabled
+  const envEnabled = process.env.NEW_RELIC_NATIVE_METRICS_ENABLED
+
+  return inputEnabled !== undefined || envEnabled !== undefined
+}
 
 /**
  * Returns true if distributed tracing has been manually enabled via configuration file or
@@ -1172,13 +1157,12 @@ Config.prototype._nativeMetricsManuallySet =
  * @returns {boolean}
  */
 Config.prototype._DTManuallySet = function _DTManuallySet(inputConfig) {
-  const inputEnabled = inputConfig
-  && inputConfig.distributed_tracing
-  && inputConfig.distributed_tracing.enabled
+  const inputEnabled =
+    inputConfig && inputConfig.distributed_tracing && inputConfig.distributed_tracing.enabled
 
   const envEnabled = process.env.NEW_RELIC_DISTRIBUTED_TRACING_ENABLED
 
-  return (inputEnabled !== undefined) || (envEnabled !== undefined)
+  return inputEnabled !== undefined || envEnabled !== undefined
 }
 
 /**
@@ -1201,7 +1185,7 @@ Config.prototype._enforceServerless = function _enforceServerless(inputConfig) {
       const name = process.env.AWS_LAMBDA_FUNCTION_NAME || 'Serverless Application'
       this.app_name = [name]
 
-      logger.info('Auto-naming serverless application to [\'%s\'] from: %s', name, namingSource)
+      logger.info("Auto-naming serverless application to ['%s'] from: %s", name, namingSource)
     }
 
     // Explicitly disable old CAT in serverless_mode
@@ -1221,26 +1205,25 @@ Config.prototype._enforceServerless = function _enforceServerless(inputConfig) {
 
       logger.info(
         'Logging is disabled by default when serverless_mode is enabled. ' +
-        'If desired, enable logging via config file or environment variable and ' +
-        'set filepath to a valid path for current environment, stdout or stderr.'
+          'If desired, enable logging via config file or environment variable and ' +
+          'set filepath to a valid path for current environment, stdout or stderr.'
       )
     }
 
-    if (this._nativeMetricsManuallySet(inputConfig) &&
-      this.plugins.native_metrics.enabled) {
+    if (this._nativeMetricsManuallySet(inputConfig) && this.plugins.native_metrics.enabled) {
       logger.info(
         'Enabling the native-metrics module when in serverless mode may greatly ' +
-        'increase cold-start times. Given the limited benefit of the VM metrics' +
-        'and general lack of control in a serverless environment, we do not ' +
-        'recommend this trade-off.'
+          'increase cold-start times. Given the limited benefit of the VM metrics' +
+          'and general lack of control in a serverless environment, we do not ' +
+          'recommend this trade-off.'
       )
     } else {
       this.plugins.native_metrics.enabled = false
 
       logger.info(
         'The native-metrics module is disabled by default when serverless_mode ' +
-        'is enabled.  If desired, enable the native-metrics module via config file ' +
-        'or environment variable.'
+          'is enabled.  If desired, enable the native-metrics module via config file ' +
+          'or environment variable.'
       )
     }
 
@@ -1252,8 +1235,8 @@ Config.prototype._enforceServerless = function _enforceServerless(inputConfig) {
       if (this.distributed_tracing.enabled) {
         logger.warn(
           'Using distributed tracing in serverless mode requires account_id be ' +
-          'defined, either in your newrelic.js file or via environment variables. ' +
-          'Disabling distributed tracing.'
+            'defined, either in your newrelic.js file or via environment variables. ' +
+            'Disabling distributed tracing.'
         )
         this.distributed_tracing.enabled = false
       }
@@ -1274,8 +1257,9 @@ Config.prototype._enforceServerless = function _enforceServerless(inputConfig) {
   DT_KEYS.forEach((key) => {
     if (this[key]) {
       logger.warn(
-        key + ' was configured locally without enabling serverless_mode. ' +
-        'This local value will be ignored and set by the New Relic servers.'
+        key +
+          ' was configured locally without enabling serverless_mode. ' +
+          'This local value will be ignored and set by the New Relic servers.'
       )
       this[key] = null
     }
@@ -1321,14 +1305,11 @@ Config.prototype._canonicalize = function _canonicalize() {
     logger.warnOnce(
       'IgnoreServerConfigurationWarning',
       'The local config setting `ignore_server_configuration` has been deprecated ' +
-      'and removed as of Agent v5. Please review agent documentation or contact ' +
-      'New Relic support.'
+        'and removed as of Agent v5. Please review agent documentation or contact ' +
+        'New Relic support.'
     )
   }
 
-  this.serverless_mode.enabled = this.serverless_mode.enabled
-    && this.feature_flag.serverless_mode
-
   if (this.license_key) {
     this.license_key = this.license_key.trim()
   }
@@ -1342,7 +1323,8 @@ function _parseCodes(codes) {
       logger.warn('Failed to parse range %s', range)
       return parsed
     }
-    if (split[0] === '') { // catch negative code. ex. -7
+    if (split[0] === '') {
+      // catch negative code. ex. -7
       return parsed.push(parseInt(range, 10))
     }
     var lower = parseInt(split[0], 10)
@@ -1355,7 +1337,8 @@ function _parseCodes(codes) {
       logger.warn('Range must start with lower bound %s', range)
     } else if (lower < 0 || upper > 1000) {
       logger.warn('Range must be between 0 and 1000 %s', range)
-    } else { // success
+    } else {
+      // success
       for (var i = lower; i <= upper; i++) {
         parsed.push(i)
       }
@@ -1402,11 +1385,7 @@ Config.prototype._applyHighSecurity = function _applyHighSecurity() {
 
     if (hsValue && typeof hsValue === 'object' && !(hsValue instanceof Array)) {
       if (typeof target[key] !== 'object') {
-        logger.warn(
-          'High Security Mode: %s should be an object, found %s',
-          key,
-          target[key]
-        )
+        logger.warn('High Security Mode: %s should be an object, found %s', key, target[key])
         target[key] = Object.create(null)
       }
 
@@ -1414,12 +1393,7 @@ Config.prototype._applyHighSecurity = function _applyHighSecurity() {
     }
 
     if (target[key] !== hsValue) {
-      logger.warn(
-        'High Security Mode: %s was set to %s, coercing to %s',
-        key,
-        target[key],
-        hsValue
-      )
+      logger.warn('High Security Mode: %s was set to %s, coercing to %s', key, target[key], hsValue)
       target[key] = hsValue
       config.emit(base + key, hsValue)
     }
@@ -1481,11 +1455,11 @@ Config.prototype.applyLasp = function applyLasp(agent, policies) {
       var policyValue = policyValues[policy.enabled ? 1 : 0]
 
       // get the most secure setting between local config and the policy
-      var finalValue = settingBlock[valueName] = config._getMostSecure(
+      var finalValue = (settingBlock[valueName] = config._getMostSecure(
         name,
         localVal,
         policyValue
-      )
+      ))
       policy.enabled = policyValues.indexOf(finalValue) === 1
       obj[name] = policy
 
@@ -1514,11 +1488,14 @@ Config.prototype.applyLasp = function applyLasp(agent, policies) {
   if (missingLASP.length) {
     fatalMessage =
       'The agent did not receive one or more security policies that it ' +
-      'expected and will shut down: ' + missingLASP.join(', ') + '.'
+      'expected and will shut down: ' +
+      missingLASP.join(', ') +
+      '.'
   } else if (missingRequired.length) {
     fatalMessage =
       'The agent received one or more required security policies that it ' +
-      'does not recognize and will shut down: ' + missingRequired.join(', ') +
+      'does not recognize and will shut down: ' +
+      missingRequired.join(', ') +
       '. Please check if a newer agent version supports these policies ' +
       'or contact support.'
   }
@@ -1533,10 +1510,10 @@ Config.prototype.applyLasp = function applyLasp(agent, policies) {
 
 Config.prototype.validateFlags = function validateFlags() {
   Object.keys(this.feature_flag).forEach(function forEachFlag(key) {
-    if (feature_flag.released.indexOf(key) > -1) {
+    if (featureFlag.released.indexOf(key) > -1) {
       logger.warn('Feature flag %s has been released', key)
     }
-    if (feature_flag.unreleased.indexOf(key) > -1) {
+    if (featureFlag.unreleased.indexOf(key) > -1) {
       logger.warn('Feature flag %s has been deprecated', key)
     }
   })
@@ -1633,7 +1610,9 @@ function initialize(config) {
    */
   logger = require('../logger')
 
-  if (config) return new Config(config)
+  if (config) {
+    return new Config(config)
+  }
 
   if (isTruthular(process.env.NEW_RELIC_NO_CONFIG_FILE)) {
     logger.info('NEW_RELIC_NO_CONFIG_FILE set, deferring to environment variables.')
@@ -1643,13 +1622,16 @@ function initialize(config) {
 
   var filepath = _findConfigFile()
   if (!filepath) {
-    logger.info(['Unable to find configuration file.',
-      'If a configuration file is desired (common for non-containerized environments),',
-      `a base configuration file can be copied from ${BASE_CONFIG_PATH}`,
-      'and renamed to "newrelic.js" in the directory from which you will start',
-      'your application.',
-      'Attempting to start agent using environment variables.'
-    ].join(' '))
+    logger.info(
+      [
+        'Unable to find configuration file.',
+        'If a configuration file is desired (common for non-containerized environments),',
+        `a base configuration file can be copied from ${BASE_CONFIG_PATH}`,
+        'and renamed to "newrelic.js" in the directory from which you will start',
+        'your application.',
+        'Attempting to start agent using environment variables.'
+      ].join(' ')
+    )
 
     return createNewConfigObject(config)
   }
@@ -1660,12 +1642,14 @@ function initialize(config) {
   } catch (error) {
     logger.error(error)
 
-    logger.warn([
-      `Unable to read existing configuration file "${filepath}".`,
-      'To allow reading of the file (if desired),',
-      'please ensure the application has read access and the file is exporting valid JSON.',
-      'Attempting to start agent using enviornment variables.'
-    ].join(' '))
+    logger.warn(
+      [
+        `Unable to read existing configuration file "${filepath}".`,
+        'To allow reading of the file (if desired),',
+        'please ensure the application has read access and the file is exporting valid JSON.',
+        'Attempting to start agent using enviornment variables.'
+      ].join(' ')
+    )
   }
 
   if (!userConf) {
@@ -1686,7 +1670,9 @@ function initialize(config) {
  */
 function createNewConfigObject(config) {
   config = new Config(Object.create(null))
-  if (config.newrelic_home) delete config.newrelic_home
+  if (config.newrelic_home) {
+    delete config.newrelic_home
+  }
   return config
 }
 
@@ -1707,7 +1693,7 @@ function getOrCreateInstance() {
 
       // Config construction has potential to throw due to invalid settings.
       // This allows the agent to return a stub api without crashing the process.
-      _configInstance = Object.assign(defaultConfig(), {agent_enabled: false})
+      _configInstance = Object.assign(defaultConfig(), { agent_enabled: false })
       _configInstance.setLogger = Config.prototype.setLogger
     }
   }
@@ -1723,8 +1709,6 @@ function createInstance(config) {
   return _configInstance
 }
 
-const REMOVE_BEFORE_SEND = new Set(['attributeFilter'])
-
 /**
  * Preserve the legacy initializer, but also allow consumers to manage their
  * own configuration if they choose.
@@ -1733,6 +1717,5 @@ Config.initialize = initialize
 Config.getOrCreateInstance = getOrCreateInstance
 Config.getInstance = getInstance
 Config.createInstance = createInstance
-Config.SPAN_EVENT_LIMIT = SPAN_EVENT_LIMIT
 
 module.exports = Config