dd-trace 5.54.0 → 5.55.0

package/packages/dd-trace/src/config.js

@@ -19,6 +19,7 @@ const telemetryMetrics = require('./telemetry/metrics')
 const { isInServerlessEnvironment, getIsGCPFunction, getIsAzureFunction } = require('./serverless')
 const { ORIGIN_KEY, GRPC_CLIENT_ERROR_STATUSES, GRPC_SERVER_ERROR_STATUSES } = require('./constants')
 const { appendRules } = require('./payload-tagging/config')
+const { getEnvironmentVariable, getEnvironmentVariables } = require('./config-helper')
 
 const tracerMetrics = telemetryMetrics.manager.namespace('tracers')
 
@@ -80,10 +81,10 @@ function getFromOtelSamplerMap (otelTracesSampler, otelTracesSamplerArg) {
 }
 
 function validateOtelPropagators (propagators) {
-  if (!process.env.PROPAGATION_STYLE_EXTRACT &&
-    !process.env.PROPAGATION_STYLE_INJECT &&
-    !process.env.DD_TRACE_PROPAGATION_STYLE &&
-    process.env.OTEL_PROPAGATORS) {
+  if (!getEnvironmentVariable('PROPAGATION_STYLE_EXTRACT') &&
+    !getEnvironmentVariable('PROPAGATION_STYLE_INJECT') &&
+    !getEnvironmentVariable('DD_TRACE_PROPAGATION_STYLE') &&
+    getEnvironmentVariable('OTEL_PROPAGATORS')) {
     for (const style in propagators) {
       if (!VALID_PROPAGATION_STYLES.has(style)) {
         log.warn('unexpected value for OTEL_PROPAGATORS environment variable')
@@ -94,7 +95,7 @@ function validateOtelPropagators (propagators) {
 }
 
 function validateEnvVarType (envVar) {
-  const value = process.env[envVar]
+  const value = getEnvironmentVariable(envVar)
   switch (envVar) {
     case 'OTEL_LOG_LEVEL':
       return VALID_LOG_LEVELS.has(value)
@@ -103,7 +104,7 @@ function validateEnvVarType (envVar) {
     case 'OTEL_SERVICE_NAME':
       return typeof value === 'string'
     case 'OTEL_TRACES_SAMPLER':
-      return getFromOtelSamplerMap(value, process.env.OTEL_TRACES_SAMPLER_ARG) !== undefined
+      return getFromOtelSamplerMap(value, getEnvironmentVariable('OTEL_TRACES_SAMPLER_ARG')) !== undefined
     case 'OTEL_TRACES_SAMPLER_ARG':
       return !Number.isNaN(Number.parseFloat(value))
     case 'OTEL_SDK_DISABLED':
@@ -119,12 +120,12 @@ function validateEnvVarType (envVar) {
 
 function checkIfBothOtelAndDdEnvVarSet () {
   for (const [otelEnvVar, ddEnvVar] of Object.entries(otelDdEnvMapping)) {
-    if (ddEnvVar && process.env[ddEnvVar] && process.env[otelEnvVar]) {
+    if (ddEnvVar && getEnvironmentVariable(ddEnvVar) && getEnvironmentVariable(otelEnvVar)) {
       log.warn(`both ${ddEnvVar} and ${otelEnvVar} environment variables are set`)
       getCounter('otel.env.hiding', ddEnvVar, otelEnvVar).inc()
     }
 
-    if (process.env[otelEnvVar] && !validateEnvVarType(otelEnvVar)) {
+    if (getEnvironmentVariable(otelEnvVar) && !validateEnvVarType(otelEnvVar)) {
       log.warn(`unexpected value for ${otelEnvVar} environment variable`)
       getCounter('otel.env.invalid', ddEnvVar, otelEnvVar).inc()
     }
@@ -178,7 +179,7 @@ function validateNamingVersion (versionString) {
  * @param {string | string[]} input
  */
 function splitJSONPathRules (input) {
-  if (!input) return null
+  if (!input) return
   if (Array.isArray(input)) return input
   if (input === 'all') return []
   return input.split(',')
@@ -211,7 +212,8 @@ function propagationStyle (key, option) {
   // Otherwise, fallback to env var parsing
   const envKey = `DD_TRACE_PROPAGATION_STYLE_${key.toUpperCase()}`
 
-  const envVar = coalesce(process.env[envKey], process.env.DD_TRACE_PROPAGATION_STYLE, process.env.OTEL_PROPAGATORS)
+  const envVar = coalesce(getEnvironmentVariable(envKey),
+    getEnvironmentVariable('DD_TRACE_PROPAGATION_STYLE'), getEnvironmentVariable('OTEL_PROPAGATORS'))
   if (envVar !== undefined) {
     return envVar.split(',')
       .filter(v => v !== '')
@@ -265,14 +267,11 @@ class Config {
 
     checkIfBothOtelAndDdEnvVarSet()
 
-    const DD_API_KEY = coalesce(
-      process.env.DATADOG_API_KEY,
-      process.env.DD_API_KEY
-    )
+    const DD_API_KEY = getEnvironmentVariable('DD_API_KEY')
 
-    if (process.env.DD_TRACE_PROPAGATION_STYLE && (
-      process.env.DD_TRACE_PROPAGATION_STYLE_INJECT ||
-      process.env.DD_TRACE_PROPAGATION_STYLE_EXTRACT
+    if (getEnvironmentVariable('DD_TRACE_PROPAGATION_STYLE') && (
+      getEnvironmentVariable('DD_TRACE_PROPAGATION_STYLE_INJECT') ||
+      getEnvironmentVariable('DD_TRACE_PROPAGATION_STYLE_EXTRACT')
     )) {
       log.warn(
         'Use either the DD_TRACE_PROPAGATION_STYLE environment variable or separate ' +
@@ -294,34 +293,34 @@ class Config {
     }
 
     const DD_INSTRUMENTATION_INSTALL_ID = coalesce(
-      process.env.DD_INSTRUMENTATION_INSTALL_ID,
+      getEnvironmentVariable('DD_INSTRUMENTATION_INSTALL_ID'),
       null
     )
     const DD_INSTRUMENTATION_INSTALL_TIME = coalesce(
-      process.env.DD_INSTRUMENTATION_INSTALL_TIME,
+      getEnvironmentVariable('DD_INSTRUMENTATION_INSTALL_TIME'),
       null
     )
     const DD_INSTRUMENTATION_INSTALL_TYPE = coalesce(
-      process.env.DD_INSTRUMENTATION_INSTALL_TYPE,
+      getEnvironmentVariable('DD_INSTRUMENTATION_INSTALL_TYPE'),
       null
     )
 
     const DD_TRACE_CLOUD_REQUEST_PAYLOAD_TAGGING = splitJSONPathRules(
       coalesce(
-        process.env.DD_TRACE_CLOUD_REQUEST_PAYLOAD_TAGGING,
+        getEnvironmentVariable('DD_TRACE_CLOUD_REQUEST_PAYLOAD_TAGGING'),
         options.cloudPayloadTagging?.request,
         ''
       ))
 
     const DD_TRACE_CLOUD_RESPONSE_PAYLOAD_TAGGING = splitJSONPathRules(
       coalesce(
-        process.env.DD_TRACE_CLOUD_RESPONSE_PAYLOAD_TAGGING,
+        getEnvironmentVariable('DD_TRACE_CLOUD_RESPONSE_PAYLOAD_TAGGING'),
         options.cloudPayloadTagging?.response,
         ''
       ))
 
     const DD_TRACE_CLOUD_PAYLOAD_TAGGING_MAX_DEPTH = coalesce(
-      process.env.DD_TRACE_CLOUD_PAYLOAD_TAGGING_MAX_DEPTH,
+      getEnvironmentVariable('DD_TRACE_CLOUD_PAYLOAD_TAGGING_MAX_DEPTH'),
       options.cloudPayloadTagging?.maxDepth,
       10
     )
@@ -370,17 +369,17 @@ class Config {
     if (this.gitMetadataEnabled) {
       this.repositoryUrl = removeUserSensitiveInfo(
         coalesce(
-          process.env.DD_GIT_REPOSITORY_URL,
+          getEnvironmentVariable('DD_GIT_REPOSITORY_URL'),
           this.tags[GIT_REPOSITORY_URL]
         )
       )
       this.commitSHA = coalesce(
-        process.env.DD_GIT_COMMIT_SHA,
+        getEnvironmentVariable('DD_GIT_COMMIT_SHA'),
         this.tags[GIT_COMMIT_SHA]
       )
       if (!this.repositoryUrl || !this.commitSHA) {
         const DD_GIT_PROPERTIES_FILE = coalesce(
-          process.env.DD_GIT_PROPERTIES_FILE,
+          getEnvironmentVariable('DD_GIT_PROPERTIES_FILE'),
           `${process.cwd()}/git.properties`
         )
         let gitPropertiesString
@@ -388,7 +387,7 @@ class Config {
           gitPropertiesString = fs.readFileSync(DD_GIT_PROPERTIES_FILE, 'utf8')
         } catch (e) {
           // Only log error if the user has set a git.properties path
-          if (process.env.DD_GIT_PROPERTIES_FILE) {
+          if (getEnvironmentVariable('DD_GIT_PROPERTIES_FILE')) {
             log.error('Error reading DD_GIT_PROPERTIES_FILE: %s', DD_GIT_PROPERTIES_FILE, e)
           }
         }
@@ -418,7 +417,7 @@ class Config {
     // TODO: Remove the experimental env vars as a major?
     const DD_TRACE_B3_ENABLED = coalesce(
       options.experimental && options.experimental.b3,
-      process.env.DD_TRACE_EXPERIMENTAL_B3_ENABLED,
+      getEnvironmentVariable('DD_TRACE_EXPERIMENTAL_B3_ENABLED'),
       false
     )
     const defaultPropagationStyle = ['datadog', 'tracecontext']
@@ -439,7 +438,7 @@ class Config {
       FUNCTION_NAME,
       K_SERVICE,
       WEBSITE_SITE_NAME
-    } = process.env
+    } = getEnvironmentVariables()
 
     const service = AWS_LAMBDA_FUNCTION_NAME ||
       FUNCTION_NAME || // Google Cloud Function Name set by deprecated runtimes
@@ -643,7 +642,6 @@ class Config {
       DD_API_SECURITY_SAMPLE_DELAY,
       DD_APM_TRACING_ENABLED,
       DD_APPSEC_AUTO_USER_INSTRUMENTATION_MODE,
-      DD_APPSEC_AUTOMATED_USER_EVENTS_TRACKING,
       DD_APPSEC_COLLECT_ALL_HEADERS,
       DD_APPSEC_ENABLED,
       DD_APPSEC_GRAPHQL_BLOCKED_TEMPLATE_JSON,
@@ -667,16 +665,14 @@ class Config {
       DD_CODE_ORIGIN_FOR_SPANS_EXPERIMENTAL_EXIT_SPANS_ENABLED,
       DD_DATA_STREAMS_ENABLED,
       DD_DBM_PROPAGATION_MODE,
-      DD_DOGSTATSD_HOSTNAME,
       DD_DOGSTATSD_HOST,
       DD_DOGSTATSD_PORT,
       DD_DYNAMIC_INSTRUMENTATION_ENABLED,
       DD_DYNAMIC_INSTRUMENTATION_REDACTED_IDENTIFIERS,
       DD_DYNAMIC_INSTRUMENTATION_REDACTION_EXCLUDED_IDENTIFIERS,
       DD_ENV,
-      DD_EXPERIMENTAL_API_SECURITY_ENABLED,
       DD_EXPERIMENTAL_APPSEC_STANDALONE_ENABLED,
-      DD_EXPERIMENTAL_PROFILING_ENABLED,
+      DD_PROFILING_ENABLED,
       DD_GRPC_CLIENT_ERROR_STATUSES,
       DD_GRPC_SERVER_ERROR_STATUSES,
       JEST_WORKER_ID,
@@ -703,7 +699,6 @@ class Config {
       DD_LLMOBS_ML_APP,
       DD_OPENAI_LOGS_ENABLED,
       DD_OPENAI_SPAN_CHAR_LIMIT,
-      DD_PROFILING_ENABLED,
       DD_PROFILING_EXPORTERS,
       DD_PROFILING_SOURCE_MAP,
       DD_INTERNAL_PROFILING_LONG_LIVED_THRESHOLD,
@@ -712,7 +707,6 @@ class Config {
       DD_RUNTIME_METRICS_ENABLED,
       DD_SERVICE,
       DD_SERVICE_MAPPING,
-      DD_SERVICE_NAME,
       DD_SITE,
       DD_SPAN_SAMPLING_RULES,
       DD_SPAN_SAMPLING_RULES_FILE,
@@ -724,7 +718,6 @@ class Config {
       DD_TELEMETRY_METRICS_ENABLED,
       DD_TRACE_128_BIT_TRACEID_GENERATION_ENABLED,
       DD_TRACE_128_BIT_TRACEID_LOGGING_ENABLED,
-      DD_TRACE_AGENT_HOSTNAME,
       DD_TRACE_AGENT_PORT,
       DD_TRACE_AGENT_PROTOCOL_VERSION,
       DD_TRACE_AWS_ADD_SPAN_POINTERS,
@@ -763,7 +756,6 @@ class Config {
       DD_TRACE_SPAN_LEAK_DEBUG,
       DD_TRACE_STARTUP_LOGS,
       DD_TRACE_TAGS,
-      DD_TRACE_TELEMETRY_ENABLED,
       DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH,
       DD_TRACING_ENABLED,
       DD_VERSION,
@@ -777,7 +769,7 @@ class Config {
       OTEL_SERVICE_NAME,
       OTEL_TRACES_SAMPLER,
       OTEL_TRACES_SAMPLER_ARG
-    } = process.env
+    } = getEnvironmentVariables()
 
     const tags = {}
     const env = setHiddenProperty(this, '_env', {})
@@ -792,10 +784,7 @@ class Config {
       DD_APM_TRACING_ENABLED,
       DD_EXPERIMENTAL_APPSEC_STANDALONE_ENABLED && isFalse(DD_EXPERIMENTAL_APPSEC_STANDALONE_ENABLED)
     ))
-    this._setBoolean(env, 'appsec.apiSecurity.enabled', coalesce(
-      DD_API_SECURITY_ENABLED && isTrue(DD_API_SECURITY_ENABLED),
-      DD_EXPERIMENTAL_API_SECURITY_ENABLED && isTrue(DD_EXPERIMENTAL_API_SECURITY_ENABLED)
-    ))
+    this._setBoolean(env, 'appsec.apiSecurity.enabled', DD_API_SECURITY_ENABLED && isTrue(DD_API_SECURITY_ENABLED))
     this._setValue(env, 'appsec.apiSecurity.sampleDelay', maybeFloat(DD_API_SECURITY_SAMPLE_DELAY))
     this._setValue(env, 'appsec.blockedTemplateGraphql', maybeFile(DD_APPSEC_GRAPHQL_BLOCKED_TEMPLATE_JSON))
     this._setValue(env, 'appsec.blockedTemplateHtml', maybeFile(DD_APPSEC_HTTP_BLOCKED_TEMPLATE_HTML))
@@ -803,10 +792,7 @@ class Config {
     this._setValue(env, 'appsec.blockedTemplateJson', maybeFile(DD_APPSEC_HTTP_BLOCKED_TEMPLATE_JSON))
     this._envUnprocessed['appsec.blockedTemplateJson'] = DD_APPSEC_HTTP_BLOCKED_TEMPLATE_JSON
     this._setBoolean(env, 'appsec.enabled', DD_APPSEC_ENABLED)
-    this._setString(env, 'appsec.eventTracking.mode', coalesce(
-      DD_APPSEC_AUTO_USER_INSTRUMENTATION_MODE,
-      DD_APPSEC_AUTOMATED_USER_EVENTS_TRACKING // TODO: remove in next major
-    ))
+    this._setString(env, 'appsec.eventTracking.mode', DD_APPSEC_AUTO_USER_INSTRUMENTATION_MODE)
     this._setBoolean(env, 'appsec.extendedHeadersCollection.enabled', DD_APPSEC_COLLECT_ALL_HEADERS)
     this._setBoolean(
       env,
@@ -847,7 +833,7 @@ class Config {
       DD_CODE_ORIGIN_FOR_SPANS_EXPERIMENTAL_EXIT_SPANS_ENABLED
     )
     this._setString(env, 'dbmPropagationMode', DD_DBM_PROPAGATION_MODE)
-    this._setString(env, 'dogstatsd.hostname', DD_DOGSTATSD_HOST || DD_DOGSTATSD_HOSTNAME)
+    this._setString(env, 'dogstatsd.hostname', DD_DOGSTATSD_HOST)
     this._setString(env, 'dogstatsd.port', DD_DOGSTATSD_PORT)
     this._setBoolean(env, 'dsmEnabled', DD_DATA_STREAMS_ENABLED)
     this._setBoolean(env, 'dynamicInstrumentation.enabled', DD_DYNAMIC_INSTRUMENTATION_ENABLED)
@@ -869,7 +855,7 @@ class Config {
     this._setIntegerRangeSet(env, 'grpc.client.error.statuses', DD_GRPC_CLIENT_ERROR_STATUSES)
     this._setIntegerRangeSet(env, 'grpc.server.error.statuses', DD_GRPC_SERVER_ERROR_STATUSES)
     this._setArray(env, 'headerTags', DD_TRACE_HEADER_TAGS)
-    this._setString(env, 'hostname', coalesce(DD_AGENT_HOST, DD_TRACE_AGENT_HOSTNAME))
+    this._setString(env, 'hostname', DD_AGENT_HOST)
     this._setValue(env, 'iast.dbRowsToTaint', maybeInt(DD_IAST_DB_ROWS_TO_TAINT))
     this._setBoolean(env, 'iast.deduplicationEnabled', DD_IAST_DEDUPLICATION_ENABLED)
     this._setBoolean(env, 'iast.enabled', DD_IAST_ENABLED)
@@ -915,7 +901,6 @@ class Config {
     this._setString(env, 'port', DD_TRACE_AGENT_PORT)
     const profilingEnabled = normalizeProfilingEnabledValue(
       coalesce(
-        DD_EXPERIMENTAL_PROFILING_ENABLED,
         DD_PROFILING_ENABLED,
         this._isInServerlessEnvironment() ? 'false' : undefined
       )
@@ -953,10 +938,10 @@ class Config {
     this._setSamplingRule(env, 'sampler.rules', safeJsonParse(DD_TRACE_SAMPLING_RULES))
     this._envUnprocessed['sampler.rules'] = DD_TRACE_SAMPLING_RULES
     this._setString(env, 'scope', DD_TRACE_SCOPE)
-    this._setString(env, 'service', DD_SERVICE || DD_SERVICE_NAME || tags.service || OTEL_SERVICE_NAME)
+    this._setString(env, 'service', DD_SERVICE || tags.service || OTEL_SERVICE_NAME)
     if (DD_SERVICE_MAPPING) {
       this._setValue(env, 'serviceMapping', Object.fromEntries(
-        process.env.DD_SERVICE_MAPPING.split(',').map(x => x.trim().split(':'))
+        DD_SERVICE_MAPPING.split(',').map(x => x.trim().split(':'))
       ))
     }
     this._setString(env, 'site', DD_SITE)
@@ -971,8 +956,7 @@ class Config {
     this._setTags(env, 'tags', tags)
     this._setValue(env, 'tagsHeaderMaxLength', DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH)
     this._setBoolean(env, 'telemetry.enabled', coalesce(
-      DD_TRACE_TELEMETRY_ENABLED, // for backward compatibility
-      DD_INSTRUMENTATION_TELEMETRY_ENABLED, // to comply with instrumentation telemetry specs
+      DD_INSTRUMENTATION_TELEMETRY_ENABLED,
       !(this._isInServerlessEnvironment() || JEST_WORKER_ID)
     ))
     this._setString(env, 'instrumentation_config_id', DD_INSTRUMENTATION_CONFIG_ID)
@@ -1016,7 +1000,7 @@ class Config {
     const tags = {}
     setHiddenProperty(this, '_optsUnprocessed', {})
 
-    options = setHiddenProperty(this, '_optionsArg', Object.assign({ ingestion: {} }, options, opts))
+    options = setHiddenProperty(this, '_optionsArg', { ingestion: {}, ...options, ...opts })
 
     tagger.add(tags, options.tags)
 
@@ -1181,20 +1165,19 @@ class Config {
 
   _isCiVisibilityItrEnabled () {
     return coalesce(
-      process.env.DD_CIVISIBILITY_ITR_ENABLED,
+      getEnvironmentVariable('DD_CIVISIBILITY_ITR_ENABLED'),
       true
     )
   }
 
   _getHostname () {
-    const DD_CIVISIBILITY_AGENTLESS_URL = process.env.DD_CIVISIBILITY_AGENTLESS_URL
+    const DD_CIVISIBILITY_AGENTLESS_URL = getEnvironmentVariable('DD_CIVISIBILITY_AGENTLESS_URL')
     const url = DD_CIVISIBILITY_AGENTLESS_URL
       ? new URL(DD_CIVISIBILITY_AGENTLESS_URL)
       : getAgentUrl(this._getTraceAgentUrl(), this._optionsArg)
     const DD_AGENT_HOST = coalesce(
       this._optionsArg.hostname,
-      process.env.DD_AGENT_HOST,
-      process.env.DD_TRACE_AGENT_HOSTNAME,
+      getEnvironmentVariable('DD_AGENT_HOST'),
       '127.0.0.1'
     )
     return DD_AGENT_HOST || (url && url.hostname)
@@ -1204,17 +1187,17 @@ class Config {
     const DD_TRACE_SPAN_ATTRIBUTE_SCHEMA = validateNamingVersion(
       coalesce(
         this._optionsArg.spanAttributeSchema,
-        process.env.DD_TRACE_SPAN_ATTRIBUTE_SCHEMA
+        getEnvironmentVariable('DD_TRACE_SPAN_ATTRIBUTE_SCHEMA')
       )
     )
 
     const peerServiceSet = (
       this._optionsArg.hasOwnProperty('spanComputePeerService') ||
-      process.env.hasOwnProperty('DD_TRACE_PEER_SERVICE_DEFAULTS_ENABLED')
+      getEnvironmentVariables().hasOwnProperty('DD_TRACE_PEER_SERVICE_DEFAULTS_ENABLED')
     )
     const peerServiceValue = coalesce(
       this._optionsArg.spanComputePeerService,
-      process.env.DD_TRACE_PEER_SERVICE_DEFAULTS_ENABLED
+      getEnvironmentVariable('DD_TRACE_PEER_SERVICE_DEFAULTS_ENABLED')
     )
 
     const spanComputePeerService = (
@@ -1230,14 +1213,14 @@ class Config {
 
   _isCiVisibilityGitUploadEnabled () {
     return coalesce(
-      process.env.DD_CIVISIBILITY_GIT_UPLOAD_ENABLED,
+      getEnvironmentVariable('DD_CIVISIBILITY_GIT_UPLOAD_ENABLED'),
       true
     )
   }
 
   _isCiVisibilityManualApiEnabled () {
     return coalesce(
-      process.env.DD_CIVISIBILITY_MANUAL_API_ENABLED,
+      getEnvironmentVariable('DD_CIVISIBILITY_MANUAL_API_ENABLED'),
       true
     )
   }
@@ -1248,7 +1231,7 @@ class Config {
 
     return apmTracingEnabled && coalesce(
       this._optionsArg.stats,
-      process.env.DD_TRACE_STATS_COMPUTATION_ENABLED,
+      getEnvironmentVariable('DD_TRACE_STATS_COMPUTATION_ENABLED'),
       getIsGCPFunction() || getIsAzureFunction()
     )
   }
@@ -1256,8 +1239,7 @@ class Config {
   _getTraceAgentUrl () {
     return coalesce(
       this._optionsArg.url,
-      process.env.DD_TRACE_AGENT_URL,
-      process.env.DD_TRACE_URL,
+      getEnvironmentVariable('DD_TRACE_AGENT_URL'),
       null
     )
   }
@@ -1277,7 +1259,7 @@ class Config {
       DD_TEST_MANAGEMENT_ENABLED,
       DD_TEST_MANAGEMENT_ATTEMPT_TO_FIX_RETRIES,
       DD_CIVISIBILITY_IMPACTED_TESTS_DETECTION_ENABLED
-    } = process.env
+    } = getEnvironmentVariables()
 
     if (DD_CIVISIBILITY_AGENTLESS_URL) {
       this._setValue(calc, 'url', new URL(DD_CIVISIBILITY_AGENTLESS_URL))
@@ -1454,12 +1436,7 @@ class Config {
     obj[name] = value
   }
 
-  // TODO: Report origin changes and errors to telemetry.
-  // TODO: Deeply merge configurations.
-  // TODO: Move change tracking to telemetry.
-  // for telemetry reporting, `name`s in `containers` need to be keys from:
-  // https://github.com/DataDog/dd-go/blob/prod/trace/apps/tracer-telemetry-intake/telemetry-payload/static/config_norm_rules.json
-  _merge () {
+  _getContainersAndOriginsOrdered () {
     const containers = [
       this._remote,
       this._options,
@@ -1478,6 +1455,17 @@ class Config {
       'calculated',
       'default'
     ]
+
+    return { containers, origins }
+  }
+
+  // TODO: Report origin changes and errors to telemetry.
+  // TODO: Deeply merge configurations.
+  // TODO: Move change tracking to telemetry.
+  // for telemetry reporting, `name`s in `containers` need to be keys from:
+  // https://github.com/DataDog/dd-go/blob/prod/trace/apps/tracer-telemetry-intake/telemetry-payload/static/config_norm_rules.json
+  _merge () {
+    const { containers, origins } = this._getContainersAndOriginsOrdered()
     const unprocessedValues = [
       this._remoteUnprocessed,
       this._optsUnprocessed,
@@ -1514,6 +1502,18 @@ class Config {
     updateConfig(changes, this)
   }
 
+  getOrigin (name) {
+    const { containers, origins } = this._getContainersAndOriginsOrdered()
+
+    for (let i = 0; i < containers.length; i++) {
+      const container = containers[i]
+      const value = container[name]
+      if (value != null || container === this._defaults) {
+        return origins[i]
+      }
+    }
+  }
+
   // TODO: Refactor the Config class so it never produces any config objects that are incompatible with MessageChannel
   /**
    * Serializes the config object so it can be passed over a Worker Thread MessageChannel.
@@ -1536,12 +1536,12 @@ function handleOtel (tagString) {
     ?.replace(/(^|,)deployment\.environment=/, '$1env:')
     .replace(/(^|,)service\.name=/, '$1service:')
     .replace(/(^|,)service\.version=/, '$1version:')
-    .replace(/=/g, ':')
+    .replaceAll('=', ':')
 }
 
 function parseSpaceSeparatedTags (tagString) {
   if (tagString && !tagString.includes(',')) {
-    tagString = tagString.replace(/\s+/g, ',')
+    tagString = tagString.replaceAll(/\s+/g, ',')
   }
   return tagString
 }
@@ -1564,9 +1564,8 @@ function getAgentUrl (url, options) {
   if (
     !options.hostname &&
     !options.port &&
-    !process.env.DD_AGENT_HOST &&
-    !process.env.DD_TRACE_AGENT_HOSTNAME &&
-    !process.env.DD_TRACE_AGENT_PORT &&
+    !getEnvironmentVariable('DD_AGENT_HOST') &&
+    !getEnvironmentVariable('DD_TRACE_AGENT_PORT') &&
     fs.existsSync('/var/run/datadog/apm.socket')
   ) {
     return new URL('unix:///var/run/datadog/apm.socket')

package/packages/dd-trace/src/config_stable.js

@@ -1,5 +1,6 @@
 const os = require('os')
 const fs = require('fs')
+const { getEnvironmentVariable } = require('../../dd-trace/src/config-helper')
 
 class StableConfig {
   constructor () {
@@ -40,6 +41,8 @@ class StableConfig {
 
     try {
       const configurator = new libconfig.JsConfigurator()
+      // Intentionally pass through the raw environment variables for reporting.
+      // eslint-disable-next-line eslint-rules/eslint-process-env
       configurator.set_envp(Object.entries(process.env).map(([key, value]) => `${key}=${value}`))
       configurator.set_args(process.argv)
       configurator.get_configuration(localConfig.toString(), fleetConfig.toString()).forEach((entry) => {
@@ -86,11 +89,11 @@ class StableConfig {
     }
 
     // Allow overriding the paths for testing
-    if (process.env.DD_TEST_LOCAL_CONFIG_PATH !== undefined) {
-      localConfigPath = process.env.DD_TEST_LOCAL_CONFIG_PATH
+    if (getEnvironmentVariable('DD_TEST_LOCAL_CONFIG_PATH') !== undefined) {
+      localConfigPath = getEnvironmentVariable('DD_TEST_LOCAL_CONFIG_PATH')
     }
-    if (process.env.DD_TEST_FLEET_CONFIG_PATH !== undefined) {
-      fleetConfigPath = process.env.DD_TEST_FLEET_CONFIG_PATH
+    if (getEnvironmentVariable('DD_TEST_FLEET_CONFIG_PATH') !== undefined) {
+      fleetConfigPath = getEnvironmentVariable('DD_TEST_FLEET_CONFIG_PATH')
     }
 
     return { localConfigPath, fleetConfigPath }

package/packages/dd-trace/src/datastreams/fnv.js

@@ -5,7 +5,7 @@ function fnv (data, hvalInit, fnvPrime, fnvSize) {
   let hval = hvalInit
   for (const byte of data) {
     hval = (hval * fnvPrime) % fnvSize
-    hval = hval ^ BigInt(byte)
+    hval ^= BigInt(byte)
   }
   return hval
 }

package/packages/dd-trace/src/datastreams/schemas/schema_builder.js

@@ -117,12 +117,12 @@ function convertKey (key) {
 function jsonStringify (obj, indent = 2) {
   // made to stringify json exactly similar to python / java in order for hashing to be the same
   const jsonString = JSON.stringify(obj, (_, value) => value, indent)
-  return jsonString.replace(/^ +/gm, ' ') // Replace leading spaces with single space
-    .replace(/\n/g, '') // Remove newlines
-    .replace(/{ /g, '{') // Remove space after '{'
-    .replace(/ }/g, '}') // Remove space before '}'
-    .replace(/\[ /g, '[') // Remove space after '['
-    .replace(/ \]/g, ']') // Remove space before ']'
+  return jsonString.replaceAll(/^ +/gm, ' ') // Replace leading spaces with single space
+    .replaceAll('\n', '') // Remove newlines
+    .replaceAll('{ ', '{') // Remove space after '{'
+    .replaceAll(' }', '}') // Remove space before '}'
+    .replaceAll('[ ', '[') // Remove space after '['
+    .replaceAll(' ]', ']') // Remove space before ']'
 }
 
 module.exports = {

package/packages/dd-trace/src/debugger/devtools_client/breakpoints.js

@@ -170,7 +170,7 @@ async function updateBreakpointInternal (breakpoint, probe) {
   const conditionBeforeNewProbe = compileCompoundCondition([...probesAtLocation.values()])
 
   // If a probe is provided, add it to the breakpoint. If not, it's because we're removing a probe, but potentially
-  // need to update the condtion of the breakpoint.
+  // need to update the condition of the breakpoint.
   if (probe) {
     probesAtLocation.set(probe.id, probe)
     probeToLocation.set(probe.id, breakpoint.locationKey)
@@ -194,7 +194,6 @@ async function updateBreakpointInternal (breakpoint, probe) {
     } catch (err) {
       throw new Error(`Error setting breakpoint for probe ${probe.id}`, { cause: err })
     }
-    breakpoint.id = result.breakpointId
     breakpointToProbes.set(result.breakpointId, probesAtLocation)
   }
 }

package/packages/dd-trace/src/debugger/devtools_client/condition.js

@@ -105,9 +105,8 @@ function compile (node) {
       return '$dd_key'
     } else if (value === '@value') {
       return '$dd_value'
-    } else {
-      return assertIdentifier(value)
     }
+    return assertIdentifier(value)
   } else if (Array.isArray(value)) {
     const args = value.map(compile)
     switch (type) {

package/packages/dd-trace/src/debugger/devtools_client/index.js

@@ -109,6 +109,7 @@ session.on('Debugger.paused', async ({ params }) => {
       if (probe.condition !== undefined) {
         // TODO: Bundle all conditions and evaluate them in a single call
         // TODO: Handle errors
+        // eslint-disable-next-line no-await-in-loop
         const { result } = await session.post('Debugger.evaluateOnCallFrame', {
           callFrameId: params.callFrames[0].callFrameId,
           expression: probe.condition,
@@ -134,7 +135,7 @@ session.on('Debugger.paused', async ({ params }) => {
 
   const timestamp = Date.now()
 
-  let evalResults = null
+  let evalResults
   const { result } = await session.post('Debugger.evaluateOnCallFrame', {
     callFrameId: params.callFrames[0].callFrameId,
     expression: templateExpressions.length === 0

package/packages/dd-trace/src/debugger/devtools_client/send.js

@@ -9,6 +9,7 @@ const request = require('../../exporters/common/request')
 const { GIT_COMMIT_SHA, GIT_REPOSITORY_URL } = require('../../plugins/util/tags')
 const log = require('../../log')
 const { version } = require('../../../../../package.json')
+const { getEnvironmentVariable } = require('../../config-helper')
 
 module.exports = send
 
@@ -21,8 +22,8 @@ const hostname = getHostname()
 const service = config.service
 
 const ddtags = [
-  ['env', process.env.DD_ENV],
-  ['version', process.env.DD_VERSION],
+  ['env', getEnvironmentVariable('DD_ENV')],
+  ['version', getEnvironmentVariable('DD_VERSION')],
   ['debugger_version', version],
   ['host_name', hostname],
   [GIT_COMMIT_SHA, config.commitSHA],

package/packages/dd-trace/src/debugger/devtools_client/snapshot/collector.js

@@ -77,9 +77,8 @@ function getObjectProperties (subtype, objectId, opts, depth) {
     return getProxy(objectId, opts, depth)
   } else if (subtype === 'arraybuffer') {
     return getArrayBuffer(objectId, opts, depth)
-  } else {
-    return getObject(objectId, opts, depth + 1, subtype === 'array' || subtype === 'typedarray')
   }
+  return getObject(objectId, opts, depth + 1, subtype === 'array' || subtype === 'typedarray')
 }
 
 // TODO: The following extra information from `internalProperties` might be relevant to include for functions:

package/packages/dd-trace/src/debugger/devtools_client/snapshot/processor.js

@@ -129,11 +129,10 @@ function toFunctionOrClass (value, maxLength) {
     // This is a function
     // TODO: Would it make sense to detect if it's an arrow function or not?
     return toObject(value.className, value.properties, maxLength)
-  } else {
-    // This is a class
-    const className = classMatch[1].trim()
-    return { type: className ? `class ${className}` : 'class' }
   }
+  // This is a class
+  const className = classMatch[1].trim()
+  return { type: className ? `class ${className}` : 'class' }
 }
 
 function toString (str, maxLength) {

package/packages/dd-trace/src/debugger/devtools_client/snapshot/redaction.js

@@ -103,7 +103,7 @@ const REDACTED_IDENTIFIERS = new Set(
 
 function normalizeName (name, isSymbol) {
   if (isSymbol) name = name.slice(7, -1) // Remove `Symbol(` and `)`
-  return name.toLowerCase().replace(/[-_@$.]/g, '')
+  return name.toLowerCase().replaceAll(/[-_@$.]/g, '')
 }
 
 module.exports = {

package/packages/dd-trace/src/debugger/index.js

@@ -9,6 +9,7 @@ let worker = null
 let configChannel = null
 let ackId = 0
 
+// eslint-disable-next-line eslint-rules/eslint-process-env
 const { NODE_OPTIONS, ...env } = process.env
 
 module.exports = {