dd-trace 5.102.0 → 5.104.0

package/packages/dd-trace/src/plugins/util/test.js

@@ -163,8 +163,12 @@ const DD_CAPABILITIES_TEST_MANAGEMENT_DISABLE = '_dd.library_capabilities.test_m
 const DD_CAPABILITIES_TEST_MANAGEMENT_ATTEMPT_TO_FIX = '_dd.library_capabilities.test_management.attempt_to_fix'
 const DD_CAPABILITIES_FAILED_TEST_REPLAY = '_dd.library_capabilities.failed_test_replay'
 
-// Library configuration request error tag
-const DD_CI_LIBRARY_CONFIGURATION_ERROR = '_dd.ci.library_configuration_error'
+// Library configuration request error tags
+const DD_CI_LIBRARY_CONFIGURATION_ERROR_SETTINGS = '_dd.ci.library_configuration_error.settings'
+const DD_CI_LIBRARY_CONFIGURATION_ERROR_SKIPPABLE_TESTS = '_dd.ci.library_configuration_error.skippable_tests'
+const DD_CI_LIBRARY_CONFIGURATION_ERROR_KNOWN_TESTS = '_dd.ci.library_configuration_error.known_tests'
+const DD_CI_LIBRARY_CONFIGURATION_ERROR_TEST_MANAGEMENT_TESTS =
+  '_dd.ci.library_configuration_error.test_management_tests'
 
 const UNSUPPORTED_TIA_FRAMEWORKS = new Set(['playwright', 'vitest'])
 const MINIMUM_FRAMEWORK_VERSION_FOR_EFD = {
@@ -230,15 +234,120 @@ const BASE_LIKE_BRANCH_FILTER = /^(main|master|preprod|prod|dev|development|trun
  */
 function getSessionRequestErrorTags (sessionSpan) {
   const tags = sessionSpan?.context()._tags
+  const sessionRequestErrorTags = {}
   if (!tags || typeof tags !== 'object') return {}
-  if (tags[DD_CI_LIBRARY_CONFIGURATION_ERROR] === 'true') {
+  if (tags[DD_CI_LIBRARY_CONFIGURATION_ERROR_SETTINGS] === 'true') {
+    sessionRequestErrorTags[DD_CI_LIBRARY_CONFIGURATION_ERROR_SETTINGS] = 'true'
+  }
+  if (tags[DD_CI_LIBRARY_CONFIGURATION_ERROR_SKIPPABLE_TESTS] === 'true') {
+    sessionRequestErrorTags[DD_CI_LIBRARY_CONFIGURATION_ERROR_SKIPPABLE_TESTS] = 'true'
+  }
+  if (tags[DD_CI_LIBRARY_CONFIGURATION_ERROR_KNOWN_TESTS] === 'true') {
+    sessionRequestErrorTags[DD_CI_LIBRARY_CONFIGURATION_ERROR_KNOWN_TESTS] = 'true'
+  }
+  if (tags[DD_CI_LIBRARY_CONFIGURATION_ERROR_TEST_MANAGEMENT_TESTS] === 'true') {
+    sessionRequestErrorTags[DD_CI_LIBRARY_CONFIGURATION_ERROR_TEST_MANAGEMENT_TESTS] = 'true'
+  }
+  return sessionRequestErrorTags
+}
+
+/**
+ * Returns ITR skipping-enabled tags from a test session span for propagation to child events.
+ * @param {{ context: () => { _tags?: Record<string, string> } } | undefined} sessionSpan
+ * @returns {Record<string, string>}
+ */
+function getSessionItrSkippingEnabledTags (sessionSpan) {
+  const tags = sessionSpan?.context()._tags
+  if (!tags || typeof tags !== 'object') return {}
+  if (tags[TEST_ITR_SKIPPING_ENABLED] !== undefined) {
     return {
-      [DD_CI_LIBRARY_CONFIGURATION_ERROR]: 'true',
+      [TEST_ITR_SKIPPING_ENABLED]: tags[TEST_ITR_SKIPPING_ENABLED],
     }
   }
   return {}
 }
 
+/**
+ * Starts supported test optimization requests together when each feature is enabled.
+ *
+ * @param {{
+ *   isKnownTestsEnabled: boolean,
+ *   isTestManagementTestsEnabled: boolean,
+ *   isSuitesSkippingEnabled?: boolean,
+ *   getKnownTests: () => Promise<object>,
+ *   getTestManagementTests: () => Promise<object>,
+ *   getSkippableSuites?: () => Promise<object>
+ * }} options - Test optimization request factories.
+ * @returns {Promise<{
+ *   knownTestsResponse?: object,
+ *   testManagementTestsResponse?: object,
+ *   skippableSuitesResponse?: object
+ * }>}
+ */
+function getTestOptimizationRequestResults ({
+  isKnownTestsEnabled,
+  isTestManagementTestsEnabled,
+  isSuitesSkippingEnabled,
+  getKnownTests,
+  getTestManagementTests,
+  getSkippableSuites,
+}) {
+  const requestPromises = []
+  const responseNames = []
+
+  if (isKnownTestsEnabled) {
+    addTestOptimizationRequest(requestPromises, responseNames, 'knownTestsResponse', getKnownTests)
+  }
+
+  if (isTestManagementTestsEnabled) {
+    addTestOptimizationRequest(
+      requestPromises,
+      responseNames,
+      'testManagementTestsResponse',
+      getTestManagementTests
+    )
+  }
+
+  if (isSuitesSkippingEnabled && getSkippableSuites) {
+    addTestOptimizationRequest(requestPromises, responseNames, 'skippableSuitesResponse', getSkippableSuites)
+  }
+
+  if (!requestPromises.length) {
+    return Promise.resolve({})
+  }
+
+  return Promise.allSettled(requestPromises).then(requestResults => {
+    const responses = {}
+
+    for (let index = 0; index < requestResults.length; index++) {
+      const requestResult = requestResults[index]
+      responses[responseNames[index]] = requestResult.status === 'fulfilled'
+        ? requestResult.value
+        : { err: requestResult.reason }
+    }
+
+    return responses
+  })
+}
+
+/**
+ * Starts a test optimization request.
+ *
+ * @param {Promise<object>[]} requestPromises - Test optimization request promises.
+ * @param {string[]} responseNames - Response keys matching request promises.
+ * @param {string} responseName - Response key for this request.
+ * @param {() => Promise<object>} getRequest - Test optimization request factory.
+ */
+function addTestOptimizationRequest (requestPromises, responseNames, responseName, getRequest) {
+  responseNames.push(responseName)
+
+  try {
+    requestPromises.push(Promise.resolve(getRequest()))
+  } catch (err) {
+    requestPromises.push(Promise.reject(err))
+  }
+}
+
 module.exports = {
   TEST_CODE_OWNERS,
   TEST_SESSION_NAME,
@@ -350,7 +459,12 @@ module.exports = {
   TEST_MANAGEMENT_ATTEMPT_TO_FIX_PASSED,
   getLibraryCapabilitiesTags,
   getSessionRequestErrorTags,
-  DD_CI_LIBRARY_CONFIGURATION_ERROR,
+  DD_CI_LIBRARY_CONFIGURATION_ERROR_SETTINGS,
+  DD_CI_LIBRARY_CONFIGURATION_ERROR_SKIPPABLE_TESTS,
+  DD_CI_LIBRARY_CONFIGURATION_ERROR_KNOWN_TESTS,
+  DD_CI_LIBRARY_CONFIGURATION_ERROR_TEST_MANAGEMENT_TESTS,
+  getSessionItrSkippingEnabledTags,
+  getTestOptimizationRequestResults,
   checkShaDiscrepancies,
   getPullRequestDiff,
   getPullRequestBaseBranch,

package/packages/dd-trace/src/plugins/util/user-provided-git.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { getValueFromEnvSources } = require('../../config/helper')
+const getConfig = require('../../config')
 const {
   GIT_COMMIT_SHA,
   GIT_BRANCH,
@@ -46,20 +46,22 @@ function validateGitCommitSha (gitCommitSha) {
 }
 
 function getUserProviderGitMetadata () {
-  const DD_GIT_COMMIT_SHA = getValueFromEnvSources('DD_GIT_COMMIT_SHA')
-  const DD_GIT_BRANCH = getValueFromEnvSources('DD_GIT_BRANCH')
-  const DD_GIT_REPOSITORY_URL = getValueFromEnvSources('DD_GIT_REPOSITORY_URL')
-  const DD_GIT_TAG = getValueFromEnvSources('DD_GIT_TAG')
-  const DD_GIT_COMMIT_MESSAGE = getValueFromEnvSources('DD_GIT_COMMIT_MESSAGE')
-  const DD_GIT_COMMIT_COMMITTER_NAME = getValueFromEnvSources('DD_GIT_COMMIT_COMMITTER_NAME')
-  const DD_GIT_COMMIT_COMMITTER_EMAIL = getValueFromEnvSources('DD_GIT_COMMIT_COMMITTER_EMAIL')
-  const DD_GIT_COMMIT_COMMITTER_DATE = getValueFromEnvSources('DD_GIT_COMMIT_COMMITTER_DATE')
-  const DD_GIT_COMMIT_AUTHOR_NAME = getValueFromEnvSources('DD_GIT_COMMIT_AUTHOR_NAME')
-  const DD_GIT_COMMIT_AUTHOR_EMAIL = getValueFromEnvSources('DD_GIT_COMMIT_AUTHOR_EMAIL')
-  const DD_GIT_COMMIT_AUTHOR_DATE = getValueFromEnvSources('DD_GIT_COMMIT_AUTHOR_DATE')
-  const DD_GIT_PULL_REQUEST_BASE_BRANCH = getValueFromEnvSources('DD_GIT_PULL_REQUEST_BASE_BRANCH')
-  const DD_GIT_PULL_REQUEST_BASE_BRANCH_SHA = getValueFromEnvSources('DD_GIT_PULL_REQUEST_BASE_BRANCH_SHA')
-  const DD_GIT_COMMIT_HEAD_SHA = getValueFromEnvSources('DD_GIT_COMMIT_HEAD_SHA')
+  const {
+    DD_GIT_COMMIT_SHA,
+    DD_GIT_BRANCH,
+    DD_GIT_REPOSITORY_URL,
+    DD_GIT_TAG,
+    DD_GIT_COMMIT_MESSAGE,
+    DD_GIT_COMMIT_COMMITTER_NAME,
+    DD_GIT_COMMIT_COMMITTER_EMAIL,
+    DD_GIT_COMMIT_COMMITTER_DATE,
+    DD_GIT_COMMIT_AUTHOR_NAME,
+    DD_GIT_COMMIT_AUTHOR_EMAIL,
+    DD_GIT_COMMIT_AUTHOR_DATE,
+    DD_GIT_PULL_REQUEST_BASE_BRANCH,
+    DD_GIT_PULL_REQUEST_BASE_BRANCH_SHA,
+    DD_GIT_COMMIT_HEAD_SHA,
+  } = getConfig()
 
   const branch = normalizeRef(DD_GIT_BRANCH)
   let tag = normalizeRef(DD_GIT_TAG)

package/packages/dd-trace/src/plugins/util/web.js

@@ -400,6 +400,17 @@ function addRequestTags (context, spanType) {
     }
   }
 
+  // Datadog scan/test markers, tagged unconditionally so the API endpoint
+  // reducer can keep scan/test traffic out of the API inventory.
+  const endpointScan = req.headers['x-datadog-endpoint-scan']
+  if (endpointScan !== undefined) {
+    span.setTag(`${HTTP_REQUEST_HEADERS}.x-datadog-endpoint-scan`, endpointScan)
+  }
+  const securityTest = req.headers['x-datadog-security-test']
+  if (securityTest !== undefined) {
+    span.setTag(`${HTTP_REQUEST_HEADERS}.x-datadog-security-test`, securityTest)
+  }
+
   addHeaders(context)
 }
 

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

@@ -110,7 +110,7 @@ class PrioritySampler {
    * Assigns a sampling priority to a span if not already set.
    *
    * @param {DatadogSpan} span
-   * @param {boolean} [auto=true] - Whether to use automatic sampling if no manual tags are present.
+   * @param {boolean} [auto] - Whether to use automatic sampling if no manual tags are present.
    * @returns {void}
    */
   sample (span, auto = true) {

package/packages/dd-trace/src/profiling/profiler.js

@@ -260,7 +260,7 @@ class Profiler extends EventEmitter {
     this.#lastStart = start
     if (!this.#timer || timeout !== this._timeoutInterval) {
       this.#timer = setTimeout(() => this._collect(snapshotKinds.PERIODIC), timeout)
-      this.#timer.unref()
+      this.#timer.unref?.()
     } else {
       this.#timer.refresh()
     }

package/packages/dd-trace/src/profiling/profilers/wall.js

@@ -223,7 +223,7 @@ class NativeWallProfiler {
       asyncContextsLiveGauge.mark(totalAsyncContextCount)
       asyncContextsUsedGauge.mark(usedAsyncContextCount)
     }, this.#telemetryHeartbeatIntervalMillis)
-    this._contextCountGaugeUpdater.unref()
+    this._contextCountGaugeUpdater.unref?.()
   }
 
   #enter () {

package/packages/dd-trace/src/profiling/ssi-heuristics.js

@@ -39,7 +39,7 @@ class SSIHeuristics {
     setTimeout(() => {
       this.shortLived = false
       this._maybeTriggered()
-    }, this.longLivedThreshold).unref()
+    }, this.longLivedThreshold).unref?.()
 
     this._onSpanCreated = this._onSpanCreated.bind(this)
     dc.subscribe('dd-trace:span:start', this._onSpanCreated)

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

@@ -5,7 +5,7 @@ const limiter = require('../../../vendor/dist/limiter')
 class RateLimiter {
   /**
    * @param {number} rateLimit - Allowed units per interval. Negative means unlimited, 0 disables.
-   * @param {'second'|'minute'|'hour'|'day'} [interval='second'] - Time window for the limiter.
+   * @param {'second'|'minute'|'hour'|'day'} [interval] - Time window for the limiter.
    */
   constructor (rateLimit, interval = 'second') {
     // TODO: Change rateLimit to integers. Right now these are sometimes strings, sometimes numbers.

package/packages/dd-trace/src/remote_config/scheduler.js

@@ -17,7 +17,7 @@ class Scheduler {
   runAfterDelay (interval = this._interval) {
     this._timer = setTimeout(this._callback, interval, () => this.runAfterDelay())
 
-    this._timer.unref()
+    this._timer.unref?.()
   }
 
   stop () {

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

@@ -31,6 +31,7 @@ const builtinModules = new Set(Module.builtinModules.map(stripNodePrefix))
 
 function isBuiltinModuleName (name) {
   if (typeof name !== 'string') return false
+  if (name === 'electron') return true
   return builtinModules.has(stripNodePrefix(name))
 }
 
@@ -101,7 +102,7 @@ function Hook (modules, options, onrequire) {
     if (cache[moduleId]) {
       // require.cache was potentially altered externally
       const cacheEntry = require.cache[filename]
-      if (cacheEntry && cacheEntry.exports !== cache[filename].original) {
+      if (cacheEntry && cacheEntry.exports !== cache[moduleId].original) {
         return cacheEntry.exports
       }
 

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

@@ -8,14 +8,9 @@ const process = require('process')
 const { performance, PerformanceObserver, monitorEventLoopDelay } = require('perf_hooks')
 const { DogStatsDClient, MetricsAggregationClient } = require('../dogstatsd')
 const log = require('../log')
-const { getValueFromEnvSources } = require('../config/helper')
 
 const { NODE_MAJOR } = require('../../../../version')
 const processTags = require('../process-tags')
-// TODO: This environment variable may not be changed, since the agent expects a flush every ten seconds.
-// It is only a variable for testing. Think about alternatives.
-const DD_RUNTIME_METRICS_FLUSH_INTERVAL = getValueFromEnvSources('DD_RUNTIME_METRICS_FLUSH_INTERVAL') ?? '10000'
-const INTERVAL = Number.parseInt(DD_RUNTIME_METRICS_FLUSH_INTERVAL, 10)
 
 const eventLoopDelayResolution = 4
 
@@ -40,6 +35,8 @@ module.exports = {
    */
   start (config) {
     this.stop()
+    // The agent expects a flush every ten seconds, so this is for tests only.
+    const flushIntervalMs = config.DD_RUNTIME_METRICS_FLUSH_INTERVAL
     const clientConfig = DogStatsDClient.generateClientConfig(config)
 
     if (config.DD_EXPERIMENTAL_PROPAGATE_PROCESS_TAGS_ENABLED) {
@@ -81,7 +78,7 @@ module.exports = {
         captureNativeMetrics(trackEventLoop, trackGc)
         captureCommonMetrics(trackEventLoop)
         client.flush()
-      }, INTERVAL)
+      }, flushIntervalMs)
     } else {
       lastCpuUsage = process.cpuUsage()
 
@@ -103,10 +100,10 @@ module.exports = {
           captureEventLoopDelay()
         }
         client.flush()
-      }, INTERVAL)
+      }, flushIntervalMs)
     }
 
-    interval.unref()
+    interval.unref?.()
   },
 
   stop () {

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

@@ -2,10 +2,12 @@
 
 const { storage } = require('../../datadog-core')
 
+const legacyStorage = storage('legacy')
+
 // TODO: refactor bind to use shimmer once the new internal tracer lands
 class Scope {
   active () {
-    const store = storage('legacy').getStore()
+    const store = legacyStorage.getStore()
 
     return store?.span ?? null
   }
@@ -13,10 +15,10 @@ class Scope {
   activate (span, callback) {
     if (typeof callback !== 'function') return callback
 
-    const oldStore = storage('legacy').getStore()
-    const newStore = span ? storage('legacy').getStore(span._store) : oldStore
+    const oldStore = legacyStorage.getStore()
+    const newStore = span ? legacyStorage.getStore(span._store) : oldStore
 
-    storage('legacy').enterWith({ ...newStore, span })
+    legacyStorage.enterWith({ ...newStore, span })
 
     try {
       return callback()
@@ -27,7 +29,7 @@ class Scope {
 
       throw e
     } finally {
-      storage('legacy').enterWith(oldStore)
+      legacyStorage.enterWith(oldStore)
     }
   }
 

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

@@ -17,10 +17,13 @@ function getIsGCPFunction () {
 /**
  * Enable GCP Pub/Sub PUSH subscription tracing for Cloud Run (K_SERVICE present).
  * PUSH: GCP sends HTTP POST requests to the service with message data in headers.
+ *
+ * Stays on the env helper to avoid closing the
+ * `config -> serverless -> config` import cycle.
  */
 function enableGCPPubSubPushSubscription () {
-  const isGCPPubSubPushSubscriptionEnabled = getValueFromEnvSources('DD_TRACE_GCP_PUBSUB_PUSH_ENABLED')
-  return getEnvironmentVariable('K_SERVICE') !== undefined && !isFalse(isGCPPubSubPushSubscriptionEnabled)
+  return getEnvironmentVariable('K_SERVICE') !== undefined &&
+    !isFalse(getValueFromEnvSources('DD_TRACE_GCP_PUBSUB_PUSH_ENABLED'))
 }
 
 function getIsAzureFunction () {

package/packages/dd-trace/src/service-naming/extra-services.js

@@ -1,13 +1,26 @@
 'use strict'
 
 const maxExtraServices = 64
+/** @type {Set<string>} */
 const extraServices = new Set()
 
+// 1-element cache of the most-recent argument. The sole production caller
+// (`span_format.js`) runs per span; without the cache every redis / mysql
+// burst pays a `Set.add` hash + probe even though the value is already
+// registered.
+/** @type {string | null | undefined} */
+let lastSeenService
+
 function getExtraServices () {
   return [...extraServices]
 }
 
+/**
+ * @param {string | null} [serviceName]
+ */
 function registerExtraService (serviceName) {
+  if (serviceName === lastSeenService) return
+  lastSeenService = serviceName
   if (serviceName && extraServices.size < maxExtraServices) {
     extraServices.add(serviceName)
   }
@@ -15,6 +28,7 @@ function registerExtraService (serviceName) {
 
 function clear () {
   extraServices.clear()
+  lastSeenService = undefined
 }
 
 module.exports = {

package/packages/dd-trace/src/service-naming/schemas/v0/messaging.js

@@ -32,6 +32,14 @@ const messaging = {
       serviceName: ({ tracerService }) => `${tracerService}-azure-service-bus`,
       serviceSource: integrationSource('azure-service-bus'),
     },
+    'electron:ipc:main:send': {
+      opName: () => 'electron.main.send',
+      serviceName: identityService,
+    },
+    'electron:ipc:renderer:send': {
+      opName: () => 'electron.renderer.send',
+      serviceName: identityService,
+    },
     'google-cloud-pubsub': {
       opName: () => 'pubsub.request',
       serviceName: ({ tracerService }) => `${tracerService}-pubsub`,
@@ -79,6 +87,18 @@ const messaging = {
       serviceName: amqpServiceName,
       serviceSource: integrationSource('amqp'),
     },
+    'electron:ipc:main:receive': {
+      opName: () => 'electron.main.receive',
+      serviceName: identityService,
+    },
+    'electron:ipc:main:handle': {
+      opName: () => 'electron.main.handle',
+      serviceName: identityService,
+    },
+    'electron:ipc:renderer:receive': {
+      opName: () => 'electron.renderer.receive',
+      serviceName: identityService,
+    },
     'google-cloud-pubsub': {
       opName: () => 'pubsub.receive',
       serviceName: ({ tracerService }) => `${tracerService}-pubsub`,

package/packages/dd-trace/src/service-naming/schemas/v0/web.js

@@ -54,6 +54,10 @@ const web = {
       serviceName: httpPluginClientService,
       serviceSource: optionServiceSource,
     },
+    'electron:net:request': {
+      opName: () => 'http.request',
+      serviceName: httpPluginClientService,
+    },
   },
   server: {
     'apollo.gateway.request': {

package/packages/dd-trace/src/service-naming/schemas/v1/messaging.js

@@ -24,6 +24,14 @@ const messaging = {
       opName: () => 'azure.eventhubs.send',
       serviceName: identityService,
     },
+    'electron:ipc:main:send': {
+      opName: () => 'electron.main.send',
+      serviceName: identityService,
+    },
+    'electron:ipc:renderer:send': {
+      opName: () => 'electron.renderer.send',
+      serviceName: identityService,
+    },
     'google-cloud-pubsub': {
       opName: () => 'gcp.pubsub.send',
       serviceName: identityService,
@@ -53,6 +61,18 @@ const messaging = {
   consumer: {
     amqplib: amqpInbound,
     amqp10: amqpInbound,
+    'electron:ipc:main:receive': {
+      opName: () => 'electron.main.receive',
+      serviceName: identityService,
+    },
+    'electron:ipc:main:handle': {
+      opName: () => 'electron.main.handle',
+      serviceName: identityService,
+    },
+    'electron:ipc:renderer:receive': {
+      opName: () => 'electron.renderer.receive',
+      serviceName: identityService,
+    },
     'google-cloud-pubsub': {
       opName: () => 'gcp.pubsub.process',
       serviceName: identityService,

package/packages/dd-trace/src/service-naming/schemas/v1/web.js

@@ -48,6 +48,10 @@ const web = {
       opName: () => 'undici.request',
       serviceName: httpPluginClientService,
     },
+    'electron:net:request': {
+      opName: () => 'http.client.request',
+      serviceName: httpPluginClientService,
+    },
   },
   server: {
     'apollo.gateway.request': {

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

@@ -165,7 +165,7 @@ class SpanStatsProcessor {
 
     if (this.enabled) {
       this.timer = setInterval(this.onInterval.bind(this), interval * 1e3)
-      this.timer.unref()
+      this.timer.unref?.()
     }
   }
 

package/packages/dd-trace/src/telemetry/dependencies.js

@@ -75,7 +75,7 @@ function waitAndSend (config, application, host) {
     if (savedDependenciesToSend.size > 0) {
       waitAndSend(config, application, host)
     }
-  }).unref()
+  }).unref?.()
 }
 
 function loadAllTheLoadedModules () {

package/packages/dd-trace/src/telemetry/endpoints.js

@@ -63,7 +63,7 @@ function scheduleFlush () {
   flushScheduled = true
   // this used to be setImmediate() instead, but it was making the system test flaky
   // don't ask me why
-  setTimeout(flushAndSend).unref()
+  setTimeout(flushAndSend).unref?.()
 }
 
 /**

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

@@ -291,7 +291,7 @@ function heartbeat (config, application) {
 
     const { reqType, payload } = createPayload('app-heartbeat')
     sendData(config, application, host, reqType, payload, updateRetryData)
-  }, config.telemetry.heartbeatInterval).unref()
+  }, config.telemetry.heartbeatInterval).unref?.()
 }
 
 /** @param {import('../config/config-base')} config */
@@ -307,7 +307,7 @@ function extendedHeartbeat (config) {
       heartbeatFailedDependencies = []
     }
     sendData(config, application, host, 'app-extended-heartbeat', appPayload)
-  }, config.telemetry.extendedHeartbeatInterval).unref()
+  }, config.telemetry.extendedHeartbeatInterval).unref?.()
 }
 
 /**