dd-trace 5.56.0 → 5.57.1

package/packages/datadog-plugin-prisma/src/engine.js

@@ -0,0 +1,81 @@
+'use strict'
+
+const DatabasePlugin = require('../../dd-trace/src/plugins/database')
+const { CLIENT_PORT_KEY } = require('../../dd-trace/src/constants')
+
+const databaseDriverMapper = {
+  postgresql: {
+    type: 'sql',
+    'db.type': 'postgres'
+  },
+  mysql: {
+    type: 'sql',
+    'db.type': 'mysql'
+  },
+  mongodb: {
+    type: 'mongodb',
+    'db.type': 'mongodb'
+  },
+  sqlite: {
+    type: 'sql',
+    'db.type': 'sqlite'
+  }
+}
+
+class PrismaEngine extends DatabasePlugin {
+  static get id () { return 'prisma' }
+  static get operation () { return 'engine' }
+  static get system () { return 'prisma' }
+
+  start (ctx) {
+    const { engineSpan, allEngineSpans, childOf, dbConfig } = ctx
+    const service = this.serviceName({ pluginConfig: this.config, system: this.system })
+    const spanName = engineSpan.name.slice(14) // remove 'prisma:engine:' prefix
+    const options = {
+      childOf,
+      resource: spanName,
+      service,
+      kind: engineSpan.kind,
+      meta: {
+        prisma: {
+          name: spanName,
+          type: 'engine'
+        }
+      }
+    }
+
+    if (spanName === 'db_query') {
+      const query = engineSpan.attributes['db.query.text']
+      const originalStatement = this.maybeTruncate(query)
+      const type = databaseDriverMapper[engineSpan.attributes['db.system']]?.type
+      const dbType = databaseDriverMapper[engineSpan.attributes['db.system']]?.['db.type']
+
+      options.resource = originalStatement
+      options.type = type || engineSpan.attributes['db.system']
+      options.meta['db.type'] = dbType || engineSpan.attributes['db.system']
+      options.meta['db.instance'] = dbConfig?.database
+      options.meta['db.name'] = dbConfig?.user
+      options.meta['out.host'] = dbConfig?.host
+      options.meta[CLIENT_PORT_KEY] = dbConfig?.port
+    }
+
+    const activeSpan = this.startSpan(this.operationName({ operation: this.operation }), options)
+    activeSpan._startTime = hrTimeToUnixTimeMs(engineSpan.startTime)
+    for (const span of allEngineSpans) {
+      if (span.parentId === engineSpan.id) {
+        const startCtx = { engineSpan: span, allEngineSpans, childOf: activeSpan, dbConfig }
+        this.start(startCtx)
+      }
+    }
+    const unixEndTime = hrTimeToUnixTimeMs(engineSpan.endTime)
+    activeSpan.finish(unixEndTime)
+  }
+}
+
+// Opentelemetry time format is defined here
+// https://github.com/open-telemetry/opentelemetry-js/blob/cbc912d/api/src/common/Time.ts#L19-L30.
+function hrTimeToUnixTimeMs ([seconds, nanoseconds]) {
+  return seconds * 1000 + nanoseconds / 1e6
+}
+
+module.exports = PrismaEngine

package/packages/datadog-plugin-prisma/src/index.js

@@ -0,0 +1,22 @@
+'use strict'
+
+const CompositePlugin = require('../../dd-trace/src/plugins/composite')
+
+const PrismaClientPlugin = require('./client')
+const PrismaEnginePlugin = require('./engine')
+
+class PrismaPlugin extends CompositePlugin {
+  static get id () { return 'prisma' }
+  static get plugins () {
+    return {
+      client: PrismaClientPlugin,
+      engine: PrismaEnginePlugin
+    }
+  }
+
+  configure (config) {
+    return super.configure(config)
+  }
+}
+
+module.exports = PrismaPlugin

package/packages/datadog-plugin-protobufjs/src/schema_iterator.js

@@ -108,7 +108,7 @@ class SchemaExtractor {
     }
     for (const field of schema.fieldsArray) {
       if (!this.extractProperty(field, schemaName, field.name, builder, depth)) {
-        log.warn(`DSM: Unable to extract field with name: ${field.name} from Avro schema with name: ${schemaName}`)
+        log.warn('DSM: Unable to extract field with name: %s from Avro schema with name:', field.name, schemaName)
       }
     }
     return true

package/packages/datadog-plugin-redis/src/index.js

@@ -13,10 +13,14 @@ class RedisPlugin extends CachePlugin {
     this._spanType = 'redis'
   }
 
-  start ({ db, command, args, connectionOptions = {}, connectionName }) {
+  bindStart (ctx) {
+    const { db, command, args, connectionOptions, connectionName } = ctx
+
     const resource = command
     const normalizedCommand = command.toUpperCase()
-    if (!this.config.filter(normalizedCommand)) return this.skip()
+    if (!this.config.filter(normalizedCommand)) {
+      return { noop: true }
+    }
 
     this.startSpan({
       resource,
@@ -29,7 +33,9 @@ class RedisPlugin extends CachePlugin {
         'out.host': connectionOptions.host,
         [CLIENT_PORT_KEY]: connectionOptions.port
       }
-    })
+    }, ctx)
+
+    return ctx.currentStore
   }
 
   configure (config) {

package/packages/datadog-plugin-router/src/index.js

@@ -106,6 +106,7 @@ class RouterPlugin extends WebPlugin {
 
     const span = this.tracer.startSpan(`${this.constructor.id}.middleware`, {
       childOf,
+      integrationName: this.constructor.id,
       tags: {
         [COMPONENT]: this.constructor.id,
         'resource.name': name || '<anonymous>'

package/packages/datadog-plugin-sharedb/src/index.js

@@ -5,7 +5,9 @@ const ServerPlugin = require('../../dd-trace/src/plugins/server')
 class SharedbPlugin extends ServerPlugin {
   static get id () { return 'sharedb' }
 
-  start ({ actionName, request }) {
+  bindStart (ctx) {
+    const { actionName, request } = ctx
+
     const span = this.startSpan('sharedb.request', {
       service: this.config.service,
       resource: getReadableResourceName(actionName, request.c, request.q),
@@ -13,19 +15,25 @@ class SharedbPlugin extends ServerPlugin {
       meta: {
         'sharedb.action': actionName
       }
-    })
+    }, ctx)
 
     if (this.config.hooks && this.config.hooks.receive) {
       this.config.hooks.receive(span, request)
     }
+
+    return ctx.currentStore
   }
 
-  finish ({ request, res }) {
-    const span = this.activeSpan
+  bindFinish (ctx) {
+    const { request, res } = ctx
+
+    const span = ctx.currentStore.span
     if (this.config.hooks && this.config.hooks.reply) {
       this.config.hooks.reply(span, request, res)
     }
-    super.finish()
+    super.finish(ctx)
+
+    return ctx.parentStore
   }
 }
 

package/packages/datadog-plugin-winston/src/index.js

@@ -1,8 +1,8 @@
 'use strict'
 
-const LogPlugin = require('../../dd-trace/src/plugins/log_plugin')
+const StructuredLogPlugin = require('../../dd-trace/src/plugins/structured_log_plugin')
 
-class WinstonPlugin extends LogPlugin {
+class WinstonPlugin extends StructuredLogPlugin {
   static get id () {
     return 'winston'
   }

package/packages/dd-trace/src/appsec/channels.js

@@ -4,34 +4,39 @@ const dc = require('dc-polyfill')
 
 // TODO: use TBD naming convention
 module.exports = {
+  apolloChannel: dc.tracingChannel('datadog:apollo:request'),
+  apolloServerCoreChannel: dc.tracingChannel('datadog:apollo-server-core:request'),
   bodyParser: dc.channel('datadog:body-parser:read:finish'),
+  childProcessExecutionTracingChannel: dc.tracingChannel('datadog:child_process:execution'),
   cookieParser: dc.channel('datadog:cookie-parser:read:finish'),
-  multerParser: dc.channel('datadog:multer:read:finish'),
-  startGraphqlResolve: dc.channel('datadog:graphql:resolver:start'),
+  expressMiddlewareError: dc.channel('apm:express:middleware:error'),
+  expressProcessParams: dc.channel('datadog:express:process_params:start'),
+  expressSession: dc.channel('datadog:express-session:middleware:finish'),
+  fastifyBodyParser: dc.channel('datadog:fastify:body-parser:finish'),
+  fastifyResponseChannel: dc.channel('datadog:fastify:response:finish'),
+  fastifyQueryParams: dc.channel('datadog:fastify:query-params:finish'),
+  fastifyCookieParser: dc.channel('datadog:fastify-cookie:read:finish'),
+  fastifyPathParams: dc.channel('datadog:fastify:path-params:finish'),
+  fsOperationStart: dc.channel('apm:fs:operation:start'),
   graphqlMiddlewareChannel: dc.tracingChannel('datadog:apollo:middleware'),
-  apolloChannel: dc.tracingChannel('datadog:apollo:request'),
-  apolloServerCoreChannel: dc.tracingChannel('datadog:apollo-server-core:request'),
-  incomingHttpRequestStart: dc.channel('dd-trace:incomingHttpRequestStart'),
+  httpClientRequestStart: dc.channel('apm:http:client:request:start'),
   incomingHttpRequestEnd: dc.channel('dd-trace:incomingHttpRequestEnd'),
-  passportVerify: dc.channel('datadog:passport:verify:finish'),
-  passportUser: dc.channel('datadog:passport:deserializeUser:finish'),
-  expressSession: dc.channel('datadog:express-session:middleware:finish'),
-  queryParser: dc.channel('datadog:query:read:finish'),
-  setCookieChannel: dc.channel('datadog:iast:set-cookie'),
+  incomingHttpRequestStart: dc.channel('dd-trace:incomingHttpRequestStart'),
+  multerParser: dc.channel('datadog:multer:read:finish'),
+  mysql2OuterQueryStart: dc.channel('datadog:mysql2:outerquery:start'),
   nextBodyParsed: dc.channel('apm:next:body-parsed'),
   nextQueryParsed: dc.channel('apm:next:query-parsed'),
-  expressProcessParams: dc.channel('datadog:express:process_params:start'),
-  routerParam: dc.channel('datadog:router:param:start'),
+  passportUser: dc.channel('datadog:passport:deserializeUser:finish'),
+  passportVerify: dc.channel('datadog:passport:verify:finish'),
+  pgPoolQueryStart: dc.channel('datadog:pg:pool:query:start'),
+  pgQueryStart: dc.channel('apm:pg:query:start'),
+  queryParser: dc.channel('datadog:query:read:finish'),
   responseBody: dc.channel('datadog:express:response:json:start'),
-  responseWriteHead: dc.channel('apm:http:server:response:writeHead:start'),
-  httpClientRequestStart: dc.channel('apm:http:client:request:start'),
   responseSetHeader: dc.channel('datadog:http:server:response:set-header:start'),
+  responseWriteHead: dc.channel('apm:http:server:response:writeHead:start'),
+  routerParam: dc.channel('datadog:router:param:start'),
+  setCookieChannel: dc.channel('datadog:iast:set-cookie'),
   setUncaughtExceptionCaptureCallbackStart: dc.channel('datadog:process:setUncaughtExceptionCaptureCallback:start'),
-  pgQueryStart: dc.channel('apm:pg:query:start'),
-  pgPoolQueryStart: dc.channel('datadog:pg:pool:query:start'),
-  mysql2OuterQueryStart: dc.channel('datadog:mysql2:outerquery:start'),
-  wafRunFinished: dc.channel('datadog:waf:run:finish'),
-  fsOperationStart: dc.channel('apm:fs:operation:start'),
-  expressMiddlewareError: dc.channel('apm:express:middleware:error'),
-  childProcessExecutionTracingChannel: dc.tracingChannel('datadog:child_process:execution')
+  startGraphqlResolve: dc.channel('datadog:graphql:resolver:start'),
+  wafRunFinished: dc.channel('datadog:waf:run:finish')
 }

package/packages/dd-trace/src/appsec/iast/analyzers/unvalidated-redirect-analyzer.js

@@ -5,13 +5,17 @@ const { UNVALIDATED_REDIRECT } = require('../vulnerabilities')
 const { getNodeModulesPaths } = require('../path-line')
 const { getRanges } = require('../taint-tracking/operations')
 const {
-  HTTP_REQUEST_HEADER_VALUE,
-  HTTP_REQUEST_PATH_PARAM,
-  HTTP_REQUEST_URI
+  HTTP_REQUEST_BODY,
+  HTTP_REQUEST_PARAMETER
 } = require('../taint-tracking/source-types')
 
 const EXCLUDED_PATHS = getNodeModulesPaths('express/lib/response.js')
 
+const VULNERABLE_SOURCE_TYPES = new Set([
+  HTTP_REQUEST_BODY,
+  HTTP_REQUEST_PARAMETER
+])
+
 class UnvalidatedRedirectAnalyzer extends InjectionAnalyzer {
   constructor () {
     super(UNVALIDATED_REDIRECT)
@@ -35,28 +39,17 @@ class UnvalidatedRedirectAnalyzer extends InjectionAnalyzer {
     if (!value) return false
 
     const ranges = getRanges(iastContext, value)
-    return ranges && ranges.length > 0 && !this._areSafeRanges(ranges)
+    return ranges?.length > 0 && this._hasUnsafeRange(ranges)
   }
 
-  // Do not report vulnerability if ranges sources are exclusively url,
-  // path params or referer header to avoid false positives.
-  _areSafeRanges (ranges) {
-    return ranges && ranges.every(
-      range => this._isPathParam(range) || this._isUrl(range) || this._isRefererHeader(range)
+  _hasUnsafeRange (ranges) {
+    return ranges.some(
+      range => this._isVulnerableRange(range)
     )
   }
 
-  _isRefererHeader (range) {
-    return range.iinfo.type === HTTP_REQUEST_HEADER_VALUE &&
-      range.iinfo.parameterName && range.iinfo.parameterName.toLowerCase() === 'referer'
-  }
-
-  _isPathParam (range) {
-    return range.iinfo.type === HTTP_REQUEST_PATH_PARAM
-  }
-
-  _isUrl (range) {
-    return range.iinfo.type === HTTP_REQUEST_URI
+  _isVulnerableRange (range) {
+    return VULNERABLE_SOURCE_TYPES.has(range.iinfo.type)
   }
 
   _getExcludedPaths () {

package/packages/dd-trace/src/appsec/iast/taint-tracking/source-types.js

@@ -7,7 +7,6 @@ module.exports = {
   HTTP_REQUEST_HEADER_NAME: 'http.request.header.name',
   HTTP_REQUEST_HEADER_VALUE: 'http.request.header',
   HTTP_REQUEST_PARAMETER: 'http.request.parameter',
-  HTTP_REQUEST_PATH: 'http.request.path',
   HTTP_REQUEST_PATH_PARAM: 'http.request.path.parameter',
   HTTP_REQUEST_URI: 'http.request.uri',
   KAFKA_MESSAGE_KEY: 'kafka.message.key',

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

@@ -7,6 +7,8 @@ const {
   bodyParser,
   cookieParser,
   multerParser,
+  fastifyBodyParser,
+  fastifyCookieParser,
   incomingHttpRequestStart,
   incomingHttpRequestEnd,
   passportVerify,
@@ -16,10 +18,13 @@ const {
   nextBodyParsed,
   nextQueryParsed,
   expressProcessParams,
+  fastifyQueryParams,
   responseBody,
   responseWriteHead,
   responseSetHeader,
-  routerParam
+  routerParam,
+  fastifyResponseChannel,
+  fastifyPathParams
 } = require('./channels')
 const waf = require('./waf')
 const addresses = require('./addresses')
@@ -77,8 +82,13 @@ function enable (_config) {
     nextBodyParsed.subscribe(onRequestBodyParsed)
     nextQueryParsed.subscribe(onRequestQueryParsed)
     expressProcessParams.subscribe(onRequestProcessParams)
+    fastifyBodyParser.subscribe(onRequestBodyParsed)
+    fastifyQueryParams.subscribe(onRequestQueryParsed)
+    fastifyCookieParser.subscribe(onRequestCookieParser)
+    fastifyPathParams.subscribe(onRequestProcessParams)
     routerParam.subscribe(onRequestProcessParams)
     responseBody.subscribe(onResponseBody)
+    fastifyResponseChannel.subscribe(onResponseBody)
     responseWriteHead.subscribe(onResponseWriteHead)
     responseSetHeader.subscribe(onResponseSetHeader)
 
@@ -367,8 +377,13 @@ function disable () {
   if (nextBodyParsed.hasSubscribers) nextBodyParsed.unsubscribe(onRequestBodyParsed)
   if (nextQueryParsed.hasSubscribers) nextQueryParsed.unsubscribe(onRequestQueryParsed)
   if (expressProcessParams.hasSubscribers) expressProcessParams.unsubscribe(onRequestProcessParams)
+  if (fastifyBodyParser.hasSubscribers) fastifyBodyParser.unsubscribe(onRequestBodyParsed)
+  if (fastifyQueryParams.hasSubscribers) fastifyQueryParams.unsubscribe(onRequestQueryParsed)
+  if (fastifyCookieParser.hasSubscribers) fastifyCookieParser.unsubscribe(onRequestCookieParser)
+  if (fastifyPathParams.hasSubscribers) fastifyPathParams.unsubscribe(onRequestProcessParams)
   if (routerParam.hasSubscribers) routerParam.unsubscribe(onRequestProcessParams)
   if (responseBody.hasSubscribers) responseBody.unsubscribe(onResponseBody)
+  if (fastifyResponseChannel.hasSubscribers) fastifyResponseChannel.unsubscribe(onResponseBody)
   if (responseWriteHead.hasSubscribers) responseWriteHead.unsubscribe(onResponseWriteHead)
   if (responseSetHeader.hasSubscribers) responseSetHeader.unsubscribe(onResponseSetHeader)
 }

package/packages/dd-trace/src/appsec/rasp/utils.js

@@ -58,11 +58,6 @@ function handleResult (result, req, res, abortController, config, raspRule) {
       const abortError = new DatadogRaspAbortError(req, res, blockingAction, raspRule, ruleTriggered)
       abortController.abort(abortError)
 
-      // TODO Delete this when support for node 16 is removed
-      if (!abortController.signal.reason) {
-        abortController.signal.reason = abortError
-      }
-
       return
     }
   }

package/packages/dd-trace/src/ci-visibility/dynamic-instrumentation/worker/index.js

@@ -30,7 +30,7 @@ const probeIdToBreakpointId = new Map()
 session.on('Debugger.paused', async ({ params: { hitBreakpoints: [hitBreakpoint], callFrames } }) => {
   const probe = breakpointIdToProbe.get(hitBreakpoint)
   if (!probe) {
-    log.warn(`No probe found for breakpoint ${hitBreakpoint}`)
+    log.warn('No probe found for breakpoint', hitBreakpoint)
     return session.post('Debugger.resume')
   }
 
@@ -95,13 +95,13 @@ async function addBreakpoint (probe) {
 
   const script = findScriptFromPartialPath(file)
   if (!script) {
-    log.error(`No loaded script found for ${file}`)
+    log.error('No loaded script found for', file)
     throw new Error(`No loaded script found for ${file}`)
   }
 
   const { url, scriptId, sourceMapURL, source } = script
 
-  log.warn(`Adding breakpoint at ${url}:${line}`)
+  log.warn('Adding breakpoint at %s:%s', url, line)
 
   let lineNumber = line
   let columnNumber = 0

package/packages/dd-trace/src/ci-visibility/early-flake-detection/get-known-tests.js

@@ -96,7 +96,7 @@ function getKnownTests ({
         incrementCountMetric(TELEMETRY_KNOWN_TESTS_RESPONSE_TESTS, {}, numTests)
         distributionMetric(TELEMETRY_KNOWN_TESTS_RESPONSE_BYTES, {}, res.length)
 
-        log.debug(() => `Number of received known tests: ${numTests}`)
+        log.debug('Number of received known tests:', numTests)
 
         done(null, knownTests)
       } catch (err) {

package/packages/dd-trace/src/ci-visibility/exporters/agentless/coverage-writer.js

@@ -68,7 +68,7 @@ class Writer extends BaseWriter {
         done()
         return
       }
-      log.debug(`Response from the intake: ${res}`)
+      log.debug('Response from the intake:', res)
       done()
     })
   }

package/packages/dd-trace/src/ci-visibility/exporters/agentless/di-logs-writer.js

@@ -45,7 +45,7 @@ class DynamicInstrumentationLogsWriter extends BaseWriter {
         done()
         return
       }
-      log.debug(`Response from the logs intake: ${res}`)
+      log.debug('Response from the logs intake:', res)
       done()
     })
   }

package/packages/dd-trace/src/ci-visibility/exporters/agentless/writer.js

@@ -48,7 +48,7 @@ class Writer extends BaseWriter {
     const startRequestTime = Date.now()
 
     incrementCountMetric(TELEMETRY_ENDPOINT_PAYLOAD_REQUESTS, { endpoint: 'test_cycle' })
-    distributionMetric(TELEMETRY_ENDPOINT_PAYLOAD_BYTES, { endpoint: 'test_cycle' }, data.length)
+    distributionMetric(TELEMETRY_ENDPOINT_PAYLOAD_BYTES, { endpoint: 'test_cycle' }, Buffer.byteLength(data))
 
     request(data, options, (err, res, statusCode) => {
       distributionMetric(
@@ -69,7 +69,7 @@ class Writer extends BaseWriter {
         done()
         return
       }
-      log.debug(`Response from the intake: ${res}`)
+      log.debug('Response from the intake:', res)
       done()
     })
   }

package/packages/dd-trace/src/ci-visibility/exporters/git/git_metadata.js

@@ -105,9 +105,9 @@ function getCommitsToUpload ({ url, repositoryUrl, latestCommits, isEvpProxy, ev
       incrementCountMetric(TELEMETRY_GIT_REQUESTS_SEARCH_COMMITS_ERRORS, { errorType: 'network' })
       return callback(new Error(`Can't parse commits to exclude response: ${e.message}`))
     }
-    log.debug(`There are ${alreadySeenCommits.length} commits to exclude.`)
+    log.debug('There are %s commits to exclude.', alreadySeenCommits.length)
     const commitsToInclude = latestCommits.filter((commit) => !alreadySeenCommits.includes(commit))
-    log.debug(`There are ${commitsToInclude.length} commits to include.`)
+    log.debug('There are %s commits to include.', commitsToInclude.length)
 
     if (!commitsToInclude.length) {
       return callback(null, [])
@@ -195,11 +195,11 @@ function generateAndUploadPackFiles ({
   repositoryUrl,
   headCommit
 }, callback) {
-  log.debug(`There are ${commitsToUpload.length} commits to upload`)
+  log.debug('There are %s commits to upload', commitsToUpload.length)
 
   const packFilesToUpload = generatePackFilesForCommits(commitsToUpload)
 
-  log.debug(`Uploading ${packFilesToUpload.length} packfiles.`)
+  log.debug('Uploading %s packfiles.', packFilesToUpload.length)
 
   if (!packFilesToUpload.length) {
     return callback(new Error('Failed to generate packfiles'))
@@ -253,14 +253,14 @@ function sendGitMetadata (url, { isEvpProxy, evpProxyPrefix }, configRepositoryU
     repositoryUrl = getRepositoryUrl()
   }
 
-  log.debug(`Uploading git history for repository ${repositoryUrl}`)
+  log.debug('Uploading git history for repository', repositoryUrl)
 
   if (!repositoryUrl) {
     return callback(new Error('Repository URL is empty'))
   }
 
   let latestCommits = getLatestCommits()
-  log.debug(`There were ${latestCommits.length} commits since last month.`)
+  log.debug('There were %s commits since last month.', latestCommits.length)
 
   const getOnFinishGetCommitsToUpload = (hasCheckedShallow) => (err, commitsToUpload) => {
     if (err) {

package/packages/dd-trace/src/ci-visibility/intelligent-test-runner/get-skippable-suites.js

@@ -86,10 +86,9 @@ function getSkippableSuites ({
       incrementCountMetric(TELEMETRY_ITR_SKIPPABLE_TESTS_ERRORS, { statusCode })
       done(err)
     } else {
-      let skippableSuites = []
       try {
         const parsedResponse = JSON.parse(res)
-        skippableSuites = parsedResponse
+        const skippableSuites = parsedResponse
           .data
           .filter(({ type }) => type === testLevel)
           .map(({ attributes: { suite, name } }) => {
@@ -107,7 +106,7 @@ function getSkippableSuites ({
           skippableSuites.length
         )
         distributionMetric(TELEMETRY_ITR_SKIPPABLE_TESTS_RESPONSE_BYTES, {}, res.length)
-        log.debug(() => `Number of received skippable ${testLevel}s: ${skippableSuites.length}`)
+        log.debug('Number of received skippable %ss:', testLevel, skippableSuites.length)
         done(null, skippableSuites, correlationId)
       } catch (err) {
         done(err)

package/packages/dd-trace/src/ci-visibility/requests/get-library-configuration.js

@@ -122,15 +122,15 @@ function getLibraryConfiguration ({
           isImpactedTestsEnabled
         }
 
-        log.debug(() => `Remote settings: ${JSON.stringify(settings)}`)
+        log.debug('Remote settings: %j', settings)
 
         if (getEnvironmentVariable('DD_CIVISIBILITY_DANGEROUSLY_FORCE_COVERAGE')) {
           settings.isCodeCoverageEnabled = true
-          log.debug(() => 'Dangerously set code coverage to true')
+          log.debug('Dangerously set code coverage to true')
         }
         if (getEnvironmentVariable('DD_CIVISIBILITY_DANGEROUSLY_FORCE_TEST_SKIPPING')) {
           settings.isSuitesSkippingEnabled = true
-          log.debug(() => 'Dangerously set test skipping to true')
+          log.debug('Dangerously set test skipping to true')
         }
 
         incrementCountMetric(TELEMETRY_GIT_REQUESTS_SETTINGS_RESPONSE, settings)