dd-trace 6.13.0 → 6.14.0

package/packages/dd-trace/src/opentracing/propagation/text_map.js

@@ -85,6 +85,18 @@ const percentByte = /%([0-9A-Fa-f]{2})/g
  * @property {string} [traceId]
  */
 
+/**
+ * @param {Array<string | undefined>} traceTagReplacements
+ * @param {string} key
+ * @returns {boolean}
+ */
+function hasTraceTagReplacement (traceTagReplacements, key) {
+  for (let index = 0; index < traceTagReplacements.length; index += 2) {
+    if (traceTagReplacements[index] === key) return true
+  }
+  return false
+}
+
 /**
  * @param {string | undefined} traceId
  * @param {string | undefined} spanId
@@ -314,18 +326,31 @@ class TextMapPropagator {
 
     const injectTraceContext = this.#config.apmTracingEnabled !== false ||
       hasTraceSourcePropagationTag(spanContext._trace.tags)
+    let traceTagReplacements
+    let optionalTraceTagCount = 0
+    if (injectTraceContext && injectCh.hasSubscribers && (
+      this.#hasPropagationStyle('inject', 'datadog') || this.#hasPropagationStyle('inject', 'tracecontext')
+    )) {
+      const injection = { spanContext }
+      injectCh.publish(injection)
+      traceTagReplacements = injection.traceTagReplacements
+      optionalTraceTagCount = injection.optionalTraceTagCount ?? 0
+    }
     if (injectTraceContext) {
-      injectedCarrier = this.#injectDatadog(spanContext, injectedCarrier ?? carrier) ?? injectedCarrier
+      injectedCarrier = this.#injectDatadog(
+        spanContext, injectedCarrier ?? carrier, traceTagReplacements, optionalTraceTagCount
+      ) ?? injectedCarrier
       injectedCarrier = this.#injectB3MultipleHeaders(spanContext, injectedCarrier ?? carrier) ?? injectedCarrier
       injectedCarrier = this.#injectB3SingleHeader(spanContext, injectedCarrier ?? carrier) ?? injectedCarrier
     }
     injectedCarrier = this
-      .#injectTraceparent(spanContext, injectedCarrier ?? carrier, injectTraceContext) ?? injectedCarrier
+      .#injectTraceparent(
+        spanContext, injectedCarrier ?? carrier, injectTraceContext, traceTagReplacements
+      ) ?? injectedCarrier
 
     if (injectedCarrier === undefined) return
 
     carrier = injectedCarrier
-    if (injectCh.hasSubscribers) injectCh.publish({ spanContext, carrier })
 
     // eslint-disable-next-line eslint-rules/eslint-log-printf-style
     log.debug(() => `Inject into carrier: ${JSON.stringify(pickTextMap(carrier))}.`)
@@ -359,9 +384,11 @@ class TextMapPropagator {
   /**
    * @param {DatadogSpanContext} spanContext
    * @param {Record<string, string>} [carrier]
+   * @param {Array<string | undefined>} [traceTagReplacements]
+   * @param {number} optionalTraceTagCount
    * @returns {Record<string, string> | undefined}
    */
-  #injectDatadog (spanContext, carrier) {
+  #injectDatadog (spanContext, carrier, traceTagReplacements, optionalTraceTagCount) {
     if (!this.#hasPropagationStyle('inject', 'datadog')) return
 
     carrier ??= {}
@@ -374,7 +401,7 @@ class TextMapPropagator {
     const priority = spanContext._sampling.priority
     if (Number.isInteger(priority)) writeDatadogSamplingPriority(carrier, priority.toString())
 
-    this.#injectTags(spanContext, carrier)
+    this.#injectTags(carrier, spanContext._trace.tags, traceTagReplacements, optionalTraceTagCount)
 
     return carrier
   }
@@ -449,23 +476,27 @@ class TextMapPropagator {
   }
 
   /**
-   * @param {DatadogSpanContext} spanContext
    * @param {Record<string, string>} carrier
+   * @param {Record<string, string>} traceTags
+   * @param {Array<string | undefined>} [traceTagReplacements]
+   * @param {number} optionalTraceTagCount
    * @returns {void}
    */
-  #injectTags (spanContext, carrier) {
-    const trace = spanContext._trace
-
+  #injectTags (carrier, traceTags, traceTagReplacements, optionalTraceTagCount) {
     if (this.#config.DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH === 0) {
       log.debug('Trace tag propagation is disabled, skipping injection.')
       return
     }
 
+    const maxLength = this.#config.DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH
     let header = ''
 
-    for (const key of Object.keys(trace.tags)) {
-      const value = trace.tags[key]
-      if (!value || !key.startsWith('_dd.p.')) continue
+    for (const key of Object.keys(traceTags)) {
+      const value = traceTags[key]
+      if (!value || !key.startsWith('_dd.p.') ||
+        traceTagReplacements && hasTraceTagReplacement(traceTagReplacements, key)) {
+        continue
+      }
       if (!tagKeyExpr.test(key) || !tagValueExpr.test(value)) {
         log.error('Trace tags from span are invalid, skipping injection.')
         return
@@ -475,7 +506,24 @@ class TextMapPropagator {
       header += `${key}=${value}`
     }
 
-    if (header.length > this.#config.DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH) {
+    if (traceTagReplacements) {
+      const requiredEntryCount = traceTagReplacements.length - optionalTraceTagCount * 2
+      for (let index = 0; index < traceTagReplacements.length; index += 2) {
+        const key = traceTagReplacements[index]
+        const value = traceTagReplacements[index + 1]
+        if (!value || !key.startsWith('_dd.p.')) continue
+        if (!tagKeyExpr.test(key) || !tagValueExpr.test(value)) {
+          log.error('Trace tags from span are invalid, skipping injection.')
+          return
+        }
+
+        const entry = `${header ? ',' : ''}${key}=${value}`
+        if (index >= requiredEntryCount && header.length + entry.length > maxLength) break
+        header += entry
+      }
+    }
+
+    if (header.length > maxLength) {
       log.error('Trace tags from span are too large, skipping injection.')
     } else if (header) {
       writeDatadogTags(carrier, header)
@@ -538,9 +586,10 @@ class TextMapPropagator {
    * @param {DatadogSpanContext} spanContext
    * @param {Record<string, string> | undefined} carrier
    * @param {boolean} injectTraceContext
+   * @param {Array<string | undefined>} [traceTagReplacements]
    * @returns {Record<string, string> | undefined}
    */
-  #injectTraceparent (spanContext, carrier, injectTraceContext) {
+  #injectTraceparent (spanContext, carrier, injectTraceContext, traceTagReplacements) {
     if (!this.#hasPropagationStyle('inject', 'tracecontext')) return
 
     if (!injectTraceContext) {
@@ -556,9 +605,12 @@ class TextMapPropagator {
     carrier ??= {}
     const {
       _sampling: { priority, mechanism },
-      _tracestate: ts = new TraceState(),
-      _trace: { origin, tags: traceTags },
+      _tracestate,
+      _trace: { origin },
     } = spanContext
+    const ts = traceTagReplacements
+      ? TraceState.fromString(_tracestate?.toString())
+      : _tracestate ?? new TraceState()
 
     writeTraceparent(carrier, spanContext.toTraceparent())
 
@@ -584,8 +636,9 @@ class TextMapPropagator {
         state.set('o', originValue)
       }
 
-      for (const key of Object.keys(traceTags)) {
-        const tagValueRaw = traceTags[key]
+      for (const key of Object.keys(spanContext._trace.tags)) {
+        if (traceTagReplacements && hasTraceTagReplacement(traceTagReplacements, key)) continue
+        const tagValueRaw = spanContext._trace.tags[key]
         if (!tagValueRaw || !key.startsWith('_dd.p.')) continue
 
         const tagKey = 't.' + key.slice(6)
@@ -598,6 +651,28 @@ class TextMapPropagator {
 
         state.set(tagKey, tagValue)
       }
+
+      if (traceTagReplacements) {
+        for (let index = 0; index < traceTagReplacements.length; index += 2) {
+          const key = traceTagReplacements[index]
+          if (!key.startsWith('_dd.p.')) continue
+
+          const tagKey = 't.' + key.slice(6)
+            .replaceAll(tracestateTagKeyFilter, '_')
+          const tagValueRaw = traceTagReplacements[index + 1]
+          if (!tagValueRaw) {
+            state.delete(tagKey)
+            continue
+          }
+
+          const tagValue = tagValueRaw
+            .toString()
+            .replaceAll(tracestateTagValueFilter, '_')
+            .replaceAll('=', '~')
+
+          state.set(tagKey, tagValue)
+        }
+      }
     })
 
     writeTracestate(carrier, ts.toString())

package/packages/dd-trace/src/payload-tagging/index.js

@@ -1,6 +1,7 @@
 'use strict'
 
-const rfdc = require('../../../../vendor/dist/rfdc')({ proto: false, circles: false })
+const createRfdc = require('../../../../vendor/dist/rfdc')
+const rfdc = createRfdc({ proto: false, circles: false })
 
 const {
   PAYLOAD_TAG_REQUEST_PREFIX,

package/packages/dd-trace/src/plugins/ci_plugin.js

@@ -728,12 +728,7 @@ module.exports = class CiPlugin extends Plugin {
       this._bufferWorkerTrace(missingTestSuite, trace)
       return
     }
-    const exporter = this.tracer._exporter
-    if (exporter.exportTraceWithDeferredTestSuite) {
-      exporter.exportTraceWithDeferredTestSuite(trace)
-    } else {
-      exporter.export(trace)
-    }
+    this.tracer._exporter.export(trace)
   }
 
   /**

package/packages/dd-trace/src/plugins/tracing.js

@@ -243,19 +243,21 @@ class TracingPlugin extends Plugin {
       serviceSource = undefined
     }
 
+    const tags = {
+      [COMPONENT]: component,
+      'service.name': serviceName || tracer._service,
+      'resource.name': resource,
+      'span.kind': kind,
+      'span.type': type,
+      ...meta,
+      ...metrics,
+    }
+    if (serviceSource !== undefined && !Object.hasOwn(tags, SVC_SRC_KEY)) tags[SVC_SRC_KEY] = serviceSource
+
     const span = tracer.startSpan(name, {
       startTime,
       childOf,
-      tags: {
-        [COMPONENT]: component,
-        'service.name': serviceName || tracer._service,
-        'resource.name': resource,
-        'span.kind': kind,
-        'span.type': type,
-        ...(serviceSource === undefined ? undefined : { [SVC_SRC_KEY]: serviceSource }),
-        ...meta,
-        ...metrics,
-      },
+      tags,
       integrationName: integrationName || component,
       links: childOf?._links,
     })

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

@@ -45,7 +45,9 @@ function parseEmailAndName (emailAndName) {
   }
   let name = ''
   let email = ''
-  const matchNameAndEmail = emailAndName.match(/(?:"?([^"]*)"?\s)?(?:<?(.+@[^>]+)>?)/)
+  // Untrusted git author metadata (a fork PR controls it). The anchor fixes the start position, the
+  // email classes cannot overlap around `@`, and the slice bounds the optional name part.
+  const matchNameAndEmail = emailAndName.slice(0, 1024).match(/^(?:"?([^"]*)"?\s)?<?([^\s@]+@[^\s>]+)>?/)
   if (matchNameAndEmail) {
     name = matchNameAndEmail[1]
     email = matchNameAndEmail[2]

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

@@ -69,26 +69,27 @@ function createInferredProxySpan (headers, childOf, tracer, reqCtx, traceCtx, co
 
   log.debug('Successfully extracted inferred span info %s for proxy:', proxyContext, proxyContext.proxySystemName)
 
+  const meta = {
+    service: proxyContext.domainName || tracer._config.service,
+    component: proxySpanInfo.component,
+    [SPAN_TYPE]: 'web',
+    [SPAN_KIND]: 'server',
+    [HTTP_METHOD]: proxyContext.method,
+    [HTTP_URL]: 'https://' + proxyContext.domainName + proxyContext.path,
+    stage: proxyContext.stage,
+    region: proxyContext.region,
+  }
+  if (proxyContext.resourcePath) meta[HTTP_ROUTE] = proxyContext.resourcePath
+  if (proxyContext.accountId) meta.account_id = proxyContext.accountId
+  if (proxyContext.apiId) meta.apiid = proxyContext.apiId
+  if (proxyContext.awsUser) meta.aws_user = proxyContext.awsUser
+
   const span = startSpanHelper(tracer, proxySpanInfo.spanName, {
     childOf,
     type: 'web',
     startTime: proxyContext.requestTime,
     integrationName: proxySpanInfo.component,
-    meta: {
-      service: proxyContext.domainName || tracer._config.service,
-      component: proxySpanInfo.component,
-      [SPAN_TYPE]: 'web',
-      [SPAN_KIND]: 'server',
-      [HTTP_METHOD]: proxyContext.method,
-      [HTTP_URL]: 'https://' + proxyContext.domainName + proxyContext.path,
-      stage: proxyContext.stage,
-      region: proxyContext.region,
-      ...(proxyContext.resourcePath && { [HTTP_ROUTE]: proxyContext.resourcePath }),
-      ...(proxyContext.accountId && { account_id: proxyContext.accountId }),
-      ...(proxyContext.apiId && { apiid: proxyContext.apiId }),
-      ...(proxyContext.region && { region: proxyContext.region }),
-      ...(proxyContext.awsUser && { aws_user: proxyContext.awsUser }),
-    },
+    meta,
   }, traceCtx, config)
 
   reqCtx.inferredProxySpan = span

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

@@ -108,6 +108,9 @@ const TEST_SOURCE_FILE = 'test.source.file'
 const TEST_SOURCE_START = 'test.source.start'
 const TEST_FAILURE_SCREENSHOT_UPLOADED = 'test.failure_screenshot.uploaded'
 const TEST_FAILURE_SCREENSHOT_UPLOAD_ERROR = 'test.failure_screenshot.upload_error'
+const TEST_FAILURE_VIDEO_UPLOADED = 'test.failure_video.uploaded'
+const TEST_FAILURE_VIDEO_UPLOAD_ERROR = 'test.failure_video.upload_error'
+const TEST_FAILURE_VIDEO_SCOPE = 'test.failure_video.scope'
 const LIBRARY_VERSION = 'library_version'
 const TEST_COMMAND = 'test.command'
 const TEST_MODULE = 'test.module'
@@ -127,7 +130,7 @@ const TEST_IS_MODIFIED = 'test.is_modified'
 const TEST_HAS_DYNAMIC_NAME = '_dd.has_dynamic_name'
 const CI_APP_ORIGIN = 'ciapp-test'
 // eslint-disable-next-line no-control-regex
-const TEST_OPTIMIZATION_NAME_CONTROL_RE = /[\u0000-\u0008\u000B\u000C\u000E-\u001F\u007F-\u009F]/g
+const TEST_OPTIMIZATION_NAME_CONTROL_RE = /[\u0000-\u0008\v\f\u000E-\u001F\u007F-\u009F]/g
 const TEST_OPTIMIZATION_NAME_WHITESPACE_RE = /\s+/g
 
 // Matches patterns that are almost certainly runtime-generated values in test names:
@@ -474,6 +477,9 @@ module.exports = {
   TEST_SOURCE_FILE,
   TEST_FAILURE_SCREENSHOT_UPLOADED,
   TEST_FAILURE_SCREENSHOT_UPLOAD_ERROR,
+  TEST_FAILURE_VIDEO_UPLOADED,
+  TEST_FAILURE_VIDEO_UPLOAD_ERROR,
+  TEST_FAILURE_VIDEO_SCOPE,
   CI_APP_ORIGIN,
   LIBRARY_VERSION,
   JEST_WORKER_TRACE_PAYLOAD_CODE,
@@ -1605,19 +1611,77 @@ function fromCoverageMapToCoverage (coverageMap) {
   }, {})
 }
 
+/**
+ * @param {number} code
+ * @returns {boolean}
+ */
+function isAsciiDigit (code) {
+  return code >= 0x30 && code <= 0x39
+}
+
+/**
+ * Parse a V8 stack frame into its file path and 1-based line number.
+ *
+ * Stack text includes caller-controlled function names and can be overwritten. Scan numeric suffixes
+ * right-to-left and discard V8's trailing anonymous location for eval frames.
+ * @param {string} frame
+ * @param {string} expectedPath A known path contained in the frame.
+ * @returns {{ file: string, line: number } | null}
+ */
+function parseStackFrameLocation (frame, expectedPath) {
+  let pathStart = frame.lastIndexOf(expectedPath)
+  while (frame.charCodeAt(pathStart - 1) !== 0x28 /* ( */ &&
+        !frame.startsWith('at ', pathStart - 3) &&
+        !frame.startsWith('file://', pathStart - 7)) {
+    const previousPathStart = frame.lastIndexOf(expectedPath, pathStart - 1)
+    if (previousPathStart === -1) break
+    pathStart = previousPathStart
+  }
+  const scope = frame.slice(pathStart)
+  let end = scope.length
+  if (scope.charCodeAt(end - 1) === 0x29 /* ) */) end--
+
+  while (true) {
+    let lastNumberStart = end
+    while (lastNumberStart > 0 && isAsciiDigit(scope.charCodeAt(lastNumberStart - 1))) {
+      lastNumberStart--
+    }
+    if (lastNumberStart === end || scope.charCodeAt(lastNumberStart - 1) !== 0x3A /* : */) {
+      return null
+    }
+    const colonBeforeLast = lastNumberStart - 1
+    let lineStart = colonBeforeLast
+    while (lineStart > 0 && isAsciiDigit(scope.charCodeAt(lineStart - 1))) {
+      lineStart--
+    }
+    if (lineStart < colonBeforeLast && scope.charCodeAt(lineStart - 1) === 0x3A /* : */) {
+      const fileEnd = lineStart - 1
+      const anonymousStart = fileEnd - 11
+      if (scope.charCodeAt(fileEnd - 1) === 0x3E /* > */ && anonymousStart >= 3 &&
+          scope.startsWith('<anonymous>', anonymousStart) &&
+          scope.charCodeAt(anonymousStart - 1) === 0x20 /* space */ &&
+          scope.charCodeAt(anonymousStart - 2) === 0x2C /* , */ &&
+          scope.charCodeAt(anonymousStart - 3) === 0x29 /* ) */) {
+        end = anonymousStart - 3
+        while (scope.charCodeAt(end - 1) === 0x29 /* ) */) end--
+        continue
+      }
+      return { file: scope.slice(0, fileEnd), line: Number(scope.slice(lineStart, colonBeforeLast)) }
+    }
+    return { file: scope.slice(0, colonBeforeLast), line: Number(scope.slice(lastNumberStart, end)) }
+  }
+}
+
 // Get the start line of a test by inspecting a given error's stack trace
 function getTestLineStart (err, testSuitePath) {
   if (!err.stack) {
     return null
   }
-  // From https://github.com/felixge/node-stack-trace/blob/ba06dcdb50d465cd440d84a563836e293b360427/index.js#L40
   const testFileLine = err.stack.split('\n').find(line => line.includes(testSuitePath))
-  try {
-    const testFileLineMatch = testFileLine.match(/at (?:(.+?)\s+\()?(?:(.+?):(\d+)(?::(\d+))?|([^)]+))\)?/)
-    return Number.parseInt(testFileLineMatch[3], 10) || null
-  } catch {
+  if (!testFileLine) {
     return null
   }
+  return parseStackFrameLocation(testFileLine, testSuitePath)?.line || null
 }
 
 // Get the end line of a test by inspecting a given function's source code
@@ -1640,7 +1704,7 @@ function parseAnnotations (annotations) {
     }
     const { type, description } = annotation
     if (type.startsWith('DD_TAGS')) {
-      const regex = /\[(.*?)\]/
+      const regex = /^DD_TAGS\[([^\]]*)\]/
       const match = regex.exec(type)
       let tagValue = ''
       if (match) {
@@ -1744,15 +1808,9 @@ function getFileAndLineNumberFromError (error, repositoryRoot) {
   }
 
   const topFrame = frames[topRelevantFrameIndex]
-  // Regular expression to match the file path, line number, and column number
-  const regex = /\s*at\s+(?:.*\()?(.+):(\d+):(\d+)\)?/
-  const match = topFrame.match(regex)
-
-  if (match) {
-    const filePath = match[1]
-    const lineNumber = Number(match[2])
-
-    return [filePath, lineNumber, topRelevantFrameIndex]
+  const location = parseStackFrameLocation(topFrame, repositoryRoot)
+  if (location) {
+    return [location.file, location.line, topRelevantFrameIndex]
   }
   return []
 }
@@ -2082,12 +2140,17 @@ function formatTestOptimizationList (items) {
   const more = items.length - shown.length
   const moreSuffix = more > 0 ? `\n  ... and ${more} more` : ''
 
-  return shown
-    .map(({ text, suffix }) => `  • ${truncateTestOptimizationNameMiddle(
+  let formattedItems = ''
+  let isFirstItem = true
+  for (const { text, suffix } of shown) {
+    if (!isFirstItem) formattedItems += '\n'
+    formattedItems += `  • ${truncateTestOptimizationNameMiddle(
       sanitizeTestOptimizationName(text),
       MAX_TEST_OPTIMIZATION_NAME_LENGTH
-    )}${suffix ? ` (${suffix})` : ''}`)
-    .join('\n') + moreSuffix
+    )}${suffix ? ` (${suffix})` : ''}`
+    isFirstItem = false
+  }
+  return formattedItems + moreSuffix
 }
 
 /**

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

@@ -14,7 +14,7 @@ const INT_SEGMENT = /^[1-9][0-9]+$/ // Integer of size at least 2 (>=10)
 const INT_ID_SEGMENT = /^(?=.*[0-9])[0-9._-]{3,}$/ // Mixed string with digits and delimiters
 const HEX_SEGMENT = /^(?=.*[0-9])[A-Fa-f0-9]{6,}$/ // Hexadecimal digits of size at least 6 with at least one decimal digit
 const HEX_ID_SEGMENT = /^(?=.*[0-9])[A-Fa-f0-9._-]{6,}$/ // Mixed string with hex digits and delimiters
-const STRING_SEGMENT = /(?:^.{20,}|[%&'()*+,:=@])/ // Long string or a string containing special characters
+const SPECIAL_CHARACTER_SEGMENT = /[%&'()*+,:=@]/
 
 /**
  * Extract full URL from HTTP request
@@ -167,7 +167,7 @@ function calculateHttpEndpoint (url) {
 
     if (HEX_ID_SEGMENT.test(element)) return '{param:hex_id}'
 
-    if (STRING_SEGMENT.test(element)) return '{param:str}'
+    if (element.length >= 20 || SPECIAL_CHARACTER_SEGMENT.test(element)) return '{param:str}'
 
     // No match
     return element

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

@@ -35,7 +35,6 @@ const HTTP_CLIENT_IP = tags.HTTP_CLIENT_IP
 const MANUAL_DROP = tags.MANUAL_DROP
 
 const contexts = new WeakMap()
-const requests = new WeakMap()
 
 // TODO: change this to no longer rely on creating a dummy plugin to be able to access startSpan
 function createWebPlugin (tracer, config = {}) {
@@ -129,7 +128,6 @@ const web = {
     context.tracer = tracer
     context.span = span
     context.res = res
-    requests.set(span, req)
 
     this.setConfig(req, config)
     addRequestTags(context, this.TYPE)
@@ -314,7 +312,6 @@ const web = {
     web.finishMiddleware(context)
 
     web.finishSpan(context, spanType)
-    requests.delete(context.span)
 
     finishInferredProxySpan(context)
   },
@@ -344,9 +341,6 @@ const web = {
   getContext (req) {
     return contexts.get(req)
   },
-  getRequest (span) {
-    return requests.get(span)
-  },
   setRouteOrEndpointTag (req) {
     const context = contexts.get(req)
 
@@ -511,9 +505,10 @@ function addResourceTag (context) {
 
   if (spanContext.getTag(RESOURCE_NAME)) return
 
-  const resource = [req.method, spanContext.getTag(HTTP_ROUTE)]
-    .filter(Boolean)
-    .join(' ')
+  let resource = req.method
+  const route = spanContext.getTag(HTTP_ROUTE)
+
+  if (route) resource += ` ${route}`
 
   span.setTag(RESOURCE_NAME, resource)
 }

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

@@ -222,4 +222,5 @@ function buildProfilingRuntime (config) {
 
 module.exports = {
   buildProfilingRuntime,
+  getProfilingTags,
 }

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

@@ -54,8 +54,13 @@ function strategiesToCallbackMode (strategies, callbackMode) {
  * @returns {string[]}
  */
 function buildExportCommand (exporters, tags) {
-  const tagString = [...Object.entries(tags),
-    ['snapshot', snapshotKinds.ON_OUT_OF_MEMORY]].map(([key, value]) => `${key}:${value}`).join(',')
+  let tagString = ''
+  for (const [key, value] of Object.entries(tags)) {
+    if (tagString) tagString += ','
+    tagString += `${key}:${value}`
+  }
+  if (tagString) tagString += ','
+  tagString += `snapshot:${snapshotKinds.ON_OUT_OF_MEMORY}`
   let urls = ''
   for (const exporter of exporters) {
     const url = exporter.getExportUrl()