dd-trace 5.84.0 → 5.85.0

package/packages/datadog-instrumentations/src/mocha/common.js

@@ -11,7 +11,7 @@ const patched = new WeakSet()
 // mocha-each support
 addHook({
   name: 'mocha-each',
-  versions: ['>=2.0.1']
+  versions: ['>=2.0.1'],
 }, mochaEach => {
   if (patched.has(mochaEach)) return mochaEach
 
@@ -25,7 +25,7 @@ addHook({
         parameterizedTestCh.publish({ title, params })
         it.apply(this, arguments)
       },
-      ...rest
+      ...rest,
     }
   })
 })
@@ -34,7 +34,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/suite.js'
+  file: 'lib/suite.js',
 }, (Suite) => {
   shimmer.wrap(Suite.prototype, 'addTest', addTest => function (test) {
     const callSites = getCallSites()

package/packages/datadog-instrumentations/src/mocha/main.js

@@ -13,7 +13,7 @@ const {
   getCoveredFilenamesFromCoverage,
   mergeCoverage,
   resetCoverage,
-  getIsFaultyEarlyFlakeDetection
+  getIsFaultyEarlyFlakeDetection,
 } = require('../../../dd-trace/src/plugins/util/test')
 
 const {
@@ -33,7 +33,7 @@ const {
   getTestFullName,
   getRunTestsWrapper,
   testsAttemptToFix,
-  testsStatuses
+  testsStatuses,
 } = require('./utils')
 
 require('./common')
@@ -196,7 +196,7 @@ function getOnEndHandler (isParallel) {
       isEarlyFlakeDetectionEnabled: config.isEarlyFlakeDetectionEnabled,
       isEarlyFlakeDetectionFaulty: config.isEarlyFlakeDetectionFaulty,
       isTestManagementEnabled: config.isTestManagementTestsEnabled,
-      isParallel
+      isParallel,
     })
   }
 }
@@ -204,7 +204,7 @@ function getOnEndHandler (isParallel) {
 function getExecutionConfiguration (runner, isParallel, frameworkVersion, onFinishRequest) {
   const ctx = {
     isParallel,
-    frameworkVersion
+    frameworkVersion,
   }
 
   const onReceivedSkippableSuites = ({ err, skippableSuites, itrCorrelationId: responseItrCorrelationId }) => {
@@ -220,9 +220,7 @@ function getExecutionConfiguration (runner, isParallel, frameworkVersion, onFini
 
     isSuitesSkipped = suitesToRun.length !== runner.suite.suites.length
 
-    log.debug(
-      () => `${suitesToRun.length} out of ${runner.suite.suites.length} suites are going to run.`
-    )
+    log.debug('%d out of %d suites are going to run.', suitesToRun.length, runner.suite.suites.length)
 
     runner.suite.suites = suitesToRun
 
@@ -343,7 +341,7 @@ function getExecutionConfiguration (runner, isParallel, frameworkVersion, onFini
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/mocha.js'
+  file: 'lib/mocha.js',
 }, (Mocha, frameworkVersion) => {
   shimmer.wrap(Mocha.prototype, 'run', run => function () {
     // Workers do not need to request any data, just run the tests
@@ -357,7 +355,8 @@ addHook({
 
     const runner = run.apply(this, arguments)
 
-    this.files.forEach(path => {
+    // eslint-disable-next-line unicorn/no-array-for-each
+    this.files.forEach((path) => {
       const isUnskippable = isMarkedAsUnskippable({ path })
       if (isUnskippable) {
         unskippableSuites.push(path)
@@ -383,7 +382,7 @@ addHook({
           onDone: (receivedCodeCoverage) => {
             untestedCoverage = receivedCodeCoverage
             global.run()
-          }
+          },
         })
       } else {
         global.run()
@@ -398,7 +397,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/cli/run-helpers.js'
+  file: 'lib/cli/run-helpers.js',
 }, (run) => {
   // `runMocha` is an async function
   shimmer.wrap(run, 'runMocha', runMocha => function () {
@@ -429,7 +428,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/runner.js'
+  file: 'lib/runner.js',
 }, function (Runner, frameworkVersion) {
   if (patched.has(Runner)) return Runner
 
@@ -473,7 +472,7 @@ addHook({
           testSuiteAbsolutePath: suite.file,
           isUnskippable,
           isForcedToRun,
-          itrCorrelationId
+          itrCorrelationId,
         }
         testFileToSuiteCtx.set(suite.file, ctx)
         testSuiteStartCh.runStores(ctx, () => {})
@@ -496,6 +495,7 @@ addHook({
         status = 'skip'
       } else {
         // has to check every test in the test file
+        // eslint-disable-next-line unicorn/no-array-for-each
         suitesInTestFile.forEach(suite => {
           suite.eachTest(test => {
             if (test.state === 'failed' || test.timedOut) {
@@ -510,7 +510,7 @@ addHook({
 
         testSuiteCodeCoverageCh.publish({
           coverageFiles,
-          suiteFile: suite.file
+          suiteFile: suite.file,
         })
         // We need to reset coverage to get a code coverage per suite
         // Before that, we preserve the original coverage
@@ -537,7 +537,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/runnable.js'
+  file: 'lib/runnable.js',
 }, (runnablePackage) => runnableWrapper(runnablePackage, config))
 
 function onMessage (message) {
@@ -557,7 +557,7 @@ addHook({
   // The version they use is 6.0.0:
   // https://github.com/mochajs/mocha/blob/612fa31228c695f16173ac675f40ccdf26b4cfb5/package.json#L75
   versions: ['>=6.0.0'],
-  file: 'src/WorkerHandler.js'
+  file: 'src/WorkerHandler.js',
 }, (workerHandlerPackage) => {
   shimmer.wrap(workerHandlerPackage.prototype, 'exec', exec => function (_, path) {
     if (!testFinishCh.hasSubscribers) {
@@ -607,7 +607,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=8.0.0'],
-  file: 'lib/nodejs/parallel-buffered-runner.js'
+  file: 'lib/nodejs/parallel-buffered-runner.js',
 }, (ParallelBufferedRunner, frameworkVersion) => {
   shimmer.wrap(ParallelBufferedRunner.prototype, 'run', run => function (cb, { files }) {
     if (!testFinishCh.hasSubscribers) {
@@ -646,7 +646,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=8.0.0'],
-  file: 'lib/nodejs/buffered-worker-pool.js'
+  file: 'lib/nodejs/buffered-worker-pool.js',
 }, (BufferedWorkerPoolPackage) => {
   const { BufferedWorkerPool } = BufferedWorkerPoolPackage
 
@@ -670,8 +670,8 @@ addHook({
         newWorkerArgs._ddIsKnownTestsEnabled = true
         newWorkerArgs._ddKnownTests = {
           mocha: {
-            [testPath]: testSuiteKnownTests
-          }
+            [testPath]: testSuiteKnownTests,
+          },
         }
       } else {
         config.isEarlyFlakeDetectionEnabled = false
@@ -689,9 +689,9 @@ addHook({
       newWorkerArgs._ddTestManagementTests = {
         mocha: {
           suites: {
-            [testPath]: testSuiteTestManagementTests
-          }
-        }
+            [testPath]: testSuiteTestManagementTests,
+          },
+        },
       }
     }
 
@@ -705,7 +705,7 @@ addHook({
       this,
       [
         testSuiteAbsolutePath,
-        newWorkerArgs
+        newWorkerArgs,
       ]
     )
 

package/packages/datadog-instrumentations/src/mocha/utils.js

@@ -64,11 +64,11 @@ function retryTest (test, numRetries, tags) {
   for (let retryIndex = 0; retryIndex < numRetries; retryIndex++) {
     const clonedTest = test.clone()
     suite.addTest(clonedTest)
-    tags.forEach(tag => {
+    for (const tag of tags) {
       if (tag) {
         clonedTest[tag] = true
       }
-    })
+    }
   }
 }
 
@@ -82,6 +82,7 @@ function getSuitesByTestFile (root) {
         suitesByTestFile[suite.file] = [suite]
       }
     }
+    // eslint-disable-next-line unicorn/no-array-for-each
     suite.suites.forEach(suite => {
       getSuites(suite)
     })
@@ -201,14 +202,14 @@ function getOnTestHandler (isMain) {
       _ddIsAttemptToFix: isAttemptToFix,
       _ddIsDisabled: isDisabled,
       _ddIsQuarantined: isQuarantined,
-      _ddIsModified: isModified
+      _ddIsModified: isModified,
     } = test
 
     const testInfo = {
       testName: test.fullTitle(),
       testSuiteAbsolutePath,
       title,
-      testStartLine
+      testStartLine,
     }
 
     if (!isMain) {
@@ -298,7 +299,7 @@ function getOnTestEndHandler (config) {
         attemptToFixFailed,
         isAttemptToFixRetry,
         isAtrRetry,
-        ...ctx.currentStore
+        ...ctx.currentStore,
       })
     }
   }
@@ -318,7 +319,7 @@ function getOnHookEndHandler () {
             status,
             hasBeenRetried: isMochaRetry(test),
             isLastRetry: getIsLastRetry(test),
-            ...ctx.currentStore
+            ...ctx.currentStore,
           })
         }
       }
@@ -389,14 +390,14 @@ function getOnPendingHandler () {
     const testStartLine = testToStartLine.get(test)
     const {
       file: testSuiteAbsolutePath,
-      title
+      title,
     } = test
 
     const testInfo = {
       testName: test.fullTitle(),
       testSuiteAbsolutePath,
       title,
-      testStartLine
+      testStartLine,
     }
 
     const ctx = getTestContext(test)
@@ -420,6 +421,7 @@ function getOnPendingHandler () {
 function getRunTestsWrapper (runTests, config) {
   return function (suite) {
     if (config.isTestManagementTestsEnabled) {
+      // eslint-disable-next-line unicorn/no-array-for-each
       suite.tests.forEach((test) => {
         const { isAttemptToFix, isDisabled, isQuarantined } = getTestProperties(test, config.testManagementTests)
         if (isAttemptToFix && !test.isPending()) {
@@ -443,6 +445,7 @@ function getRunTestsWrapper (runTests, config) {
     }
 
     if (config.isImpactedTestsEnabled) {
+      // eslint-disable-next-line unicorn/no-array-for-each
       suite.tests.forEach((test) => {
         isModifiedCh.publish({
           modifiedFiles: config.modifiedFiles,
@@ -458,14 +461,15 @@ function getRunTestsWrapper (runTests, config) {
                 )
               }
             }
-          }
+          },
         })
       })
     }
 
     if (config.isKnownTestsEnabled) {
       // by the time we reach `this.on('test')`, it is too late. We need to add retries here
-      suite.tests.forEach(test => {
+      // eslint-disable-next-line unicorn/no-array-for-each
+      suite.tests.forEach((test) => {
         if (!test.isPending() && isNewTest(test, config.knownTests)) {
           test._ddIsNew = true
           if (config.isEarlyFlakeDetectionEnabled && !test._ddIsAttemptToFix && !test._ddIsModified) {
@@ -506,5 +510,5 @@ module.exports = {
   newTests,
   testsQuarantined,
   testsAttemptToFix,
-  testsStatuses
+  testsStatuses,
 }

package/packages/datadog-instrumentations/src/mocha/worker.js

@@ -10,7 +10,7 @@ const {
   getOnHookEndHandler,
   getOnFailHandler,
   getOnPendingHandler,
-  getRunTestsWrapper
+  getRunTestsWrapper,
 } = require('./utils')
 require('./common')
 
@@ -21,7 +21,7 @@ const config = {}
 addHook({
   name: 'mocha',
   versions: ['>=8.0.0'],
-  file: 'lib/mocha.js'
+  file: 'lib/mocha.js',
 }, (Mocha) => {
   shimmer.wrap(Mocha.prototype, 'run', run => function () {
     if (this.options._ddIsKnownTestsEnabled) {
@@ -58,7 +58,7 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/runner.js'
+  file: 'lib/runner.js',
 }, function (Runner) {
   shimmer.wrap(Runner.prototype, 'runTests', runTests => getRunTestsWrapper(runTests, config))
 
@@ -91,6 +91,6 @@ addHook({
 addHook({
   name: 'mocha',
   versions: ['>=5.2.0'],
-  file: 'lib/runnable.js'
+  file: 'lib/runnable.js',
 }, runnableWrapper)
 // TODO: parallel mode does not support flaky test retries, so no library config is passed.

package/packages/datadog-instrumentations/src/moleculer/server.js

@@ -47,7 +47,7 @@ function createMiddleware () {
           }
         })
       })
-    }
+    },
   }
 }
 

package/packages/datadog-instrumentations/src/mongodb-core.js

@@ -4,7 +4,7 @@ const shimmer = require('../../datadog-shimmer')
 const {
   channel,
   addHook,
-  AsyncResource
+  AsyncResource,
 } = require('./helpers/instrument')
 
 const startCh = channel('apm:mongodb:query:start')
@@ -161,7 +161,7 @@ function instrument (operation, command, instance, args, server, ns, ops, option
     ns,
     ops,
     options: serverInfo,
-    name
+    name,
   }
   return startCh.runStores(ctx, () => {
     args[index] = shimmer.wrapFunction(callback, callback => function (err, res) {
@@ -193,7 +193,7 @@ function instrumentPromise (operation, command, instance, args, server, ns, ops,
     ns,
     ops,
     options: serverInfo,
-    name
+    name,
   }
 
   return startCh.runStores(ctx, () => {

package/packages/datadog-instrumentations/src/mongodb.js

@@ -5,7 +5,7 @@ require('./mongodb-core')
 const shimmer = require('../../datadog-shimmer')
 const {
   channel,
-  addHook
+  addHook,
 } = require('./helpers/instrument')
 
 // collection methods with filter
@@ -17,20 +17,20 @@ const collectionMethodsWithFilter = [
   'find',
   'findOneAndDelete',
   'findOneAndReplace',
-  'replaceOne'
+  'replaceOne',
 ] // findOne is ignored because it calls to find
 
 const collectionMethodsWithTwoFilters = [
   'findOneAndUpdate',
   'updateMany',
-  'updateOne'
+  'updateOne',
 ]
 
 const startCh = channel('datadog:mongodb:collection:filter:start')
 
 addHook({ name: 'mongodb', versions: ['>=3.3 <5', '5', '>=6'] }, mongodb => {
-  [...collectionMethodsWithFilter, ...collectionMethodsWithTwoFilters].forEach(methodName => {
-    if (!(methodName in mongodb.Collection.prototype)) return
+  for (const methodName of [...collectionMethodsWithFilter, ...collectionMethodsWithTwoFilters]) {
+    if (!(methodName in mongodb.Collection.prototype)) continue
 
     const useTwoArguments = collectionMethodsWithTwoFilters.includes(methodName)
 
@@ -42,7 +42,7 @@ addHook({ name: 'mongodb', versions: ['>=3.3 <5', '5', '>=6'] }, mongodb => {
 
         const ctx = {
           filters: [arguments[0]],
-          methodName
+          methodName,
         }
 
         if (useTwoArguments) {
@@ -54,6 +54,6 @@ addHook({ name: 'mongodb', versions: ['>=3.3 <5', '5', '>=6'] }, mongodb => {
         })
       }
     })
-  })
+  }
   return mongodb
 })

package/packages/datadog-instrumentations/src/mongoose.js

@@ -25,7 +25,7 @@ function wrapAddQueue (addQueue) {
 addHook({
   name: 'mongoose',
   versions: ['>=4.6.4 <5', '5', '6', '>=7'],
-  file: 'lib/index.js'
+  file: 'lib/index.js',
 }, mongoose => {
   // As of Mongoose 7, custom promise libraries are no longer supported and mongoose.Promise may be undefined
   if (mongoose.Promise && mongoose.Promise !== global.Promise) {
@@ -47,23 +47,23 @@ const collectionMethodsWithFilter = [
   'findOneAndDelete',
   'findOneAndReplace',
   'replaceOne',
-  'remove'
+  'remove',
 ]
 
 const collectionMethodsWithTwoFilters = [
   'findOneAndUpdate',
   'updateMany',
-  'updateOne'
+  'updateOne',
 ]
 
 addHook({
   name: 'mongoose',
   versions: ['>=4.6.4 <5', '5', '6', '>=7'],
-  file: 'lib/model.js'
+  file: 'lib/model.js',
 }, Model => {
-  [...collectionMethodsWithFilter, ...collectionMethodsWithTwoFilters].forEach(methodName => {
+  for (const methodName of [...collectionMethodsWithFilter, ...collectionMethodsWithTwoFilters]) {
     const useTwoArguments = collectionMethodsWithTwoFilters.includes(methodName)
-    if (!(methodName in Model)) return
+    if (!(methodName in Model)) continue
 
     shimmer.wrap(Model, methodName, method => {
       return function wrappedModelMethod () {
@@ -97,7 +97,7 @@ addHook({
 
         const ctx = {
           filters,
-          methodName
+          methodName,
         }
 
         return startCh.runStores(ctx, () => {
@@ -153,7 +153,7 @@ addHook({
         })
       }
     })
-  })
+  }
 
   return Model
 })
@@ -163,14 +163,14 @@ const sanitizeFilterFinishCh = channel('datadog:mongoose:sanitize-filter:finish'
 addHook({
   name: 'mongoose',
   versions: ['6', '>=7'],
-  file: 'lib/helpers/query/sanitizeFilter.js'
+  file: 'lib/helpers/query/sanitizeFilter.js',
 }, sanitizeFilter => {
   return shimmer.wrapFunction(sanitizeFilter, sanitizeFilter => function wrappedSanitizeFilter () {
     const sanitizedObject = sanitizeFilter.apply(this, arguments)
 
     if (sanitizeFilterFinishCh.hasSubscribers) {
       sanitizeFilterFinishCh.publish({
-        sanitizedObject
+        sanitizedObject,
       })
     }
 

package/packages/datadog-instrumentations/src/mquery.js

@@ -4,7 +4,7 @@ const dc = require('dc-polyfill')
 const shimmer = require('../../datadog-shimmer')
 const {
   channel,
-  addHook
+  addHook,
 } = require('./helpers/instrument')
 
 const prepareCh = channel('datadog:mquery:filter:prepare')
@@ -17,7 +17,7 @@ const methods = [
   'findOneAndDelete',
   'count',
   'distinct',
-  'where'
+  'where',
 ]
 
 const methodsOptionalArgs = ['findOneAndUpdate']
@@ -36,10 +36,10 @@ function getFilters (args, methodName) {
 
 addHook({
   name: 'mquery',
-  versions: ['>=5.0.0']
+  versions: ['>=5.0.0'],
 }, Query => {
-  [...methods, ...methodsOptionalArgs].forEach(methodName => {
-    if (!(methodName in Query.prototype)) return
+  for (const methodName of [...methods, ...methodsOptionalArgs]) {
+    if (!(methodName in Query.prototype)) continue
 
     shimmer.wrap(Query.prototype, methodName, method => {
       return function () {
@@ -53,7 +53,7 @@ addHook({
         return method.apply(this, arguments)
       }
     })
-  })
+  }
 
   shimmer.wrap(Query.prototype, 'exec', originalExec => {
     return function wrappedExec () {

package/packages/datadog-instrumentations/src/multer.js

@@ -23,7 +23,7 @@ function publishRequestBodyAndNext (req, res, next) {
 addHook({
   name: 'multer',
   file: 'lib/make-middleware.js',
-  versions: ['^1.4.4-lts.1']
+  versions: ['^1.4.4-lts.1'],
 }, makeMiddleware => {
   return shimmer.wrapFunction(makeMiddleware, makeMiddleware => function () {
     const middleware = makeMiddleware.apply(this, arguments)

package/packages/datadog-instrumentations/src/mysql.js

@@ -44,7 +44,7 @@ addHook({ name: 'mysql', file: 'lib/Connection.js', versions: ['>=2'] }, Connect
 
         return res
       } catch (err) {
-        err.stack // trigger getting the stack at the original throwing point
+        void err.stack // trigger getting the stack at the original throwing point
         ctx.error = err
         errorCh.publish(ctx)
 

package/packages/datadog-instrumentations/src/net.js

@@ -89,13 +89,13 @@ function getOptions (args) {
     case 'string':
       if (Number.isNaN(Number.parseFloat(args[0]))) {
         return {
-          path: args[0]
+          path: args[0],
         }
       }
     case 'number': // eslint-disable-line no-fallthrough
       return {
         port: args[0],
-        host: typeof args[1] === 'string' ? args[1] : 'localhost'
+        host: typeof args[1] === 'string' ? args[1] : 'localhost',
       }
   }
 }
@@ -125,9 +125,9 @@ function setupListeners (socket, protocol, ctx, finishCh, errorCh) {
 
   const cleanupOtherListeners = function () {
     socket.removeListener('connect', localListener)
-    events.forEach(event => {
+    for (const event of events) {
       socket.removeListener(event, wrapListener)
-    })
+    }
   }
 
   // TODO: Identify why the connect listener should remove the other listeners.
@@ -137,7 +137,7 @@ function setupListeners (socket, protocol, ctx, finishCh, errorCh) {
     events.push('connect')
   }
 
-  events.forEach(event => {
+  for (const event of events) {
     socket.once(event, wrapListener)
-  })
+  }
 }

package/packages/datadog-instrumentations/src/next.js

@@ -208,7 +208,7 @@ function finish (ctx, result, err) {
 addHook({
   name: 'next',
   versions: ['>=13.3.0'],
-  file: 'dist/server/web/spec-extension/adapters/next-request.js'
+  file: 'dist/server/web/spec-extension/adapters/next-request.js',
 }, NextRequestAdapter => {
   shimmer.wrap(NextRequestAdapter.NextRequestAdapter, 'fromNodeNextRequest', fromNodeNextRequest => {
     return function (nodeNextRequest) {
@@ -223,13 +223,13 @@ addHook({
 addHook({
   name: 'next',
   versions: ['>=11.1'],
-  file: 'dist/server/serve-static.js'
+  file: 'dist/server/serve-static.js',
 }, serveStatic => shimmer.wrap(serveStatic, 'serveStatic', wrapServeStatic, { replaceGetter: true }))
 
 addHook({
   name: 'next',
   versions: ['>=10.2 <11.1'],
-  file: 'dist/next-server/server/serve-static.js'
+  file: 'dist/next-server/server/serve-static.js',
 }, serveStatic => shimmer.wrap(serveStatic, 'serveStatic', wrapServeStatic, { replaceGetter: true }))
 
 addHook({ name: 'next', versions: ['>=11.1'], file: 'dist/server/next-server.js' }, nextServer => {
@@ -258,7 +258,7 @@ addHook({ name: 'next', versions: ['>=13.2'], file: 'dist/server/next-server.js'
 addHook({
   name: 'next',
   versions: ['>=11.1 <13.2'],
-  file: 'dist/server/next-server.js'
+  file: 'dist/server/next-server.js',
 }, nextServer => {
   const Server = nextServer.default
   shimmer.wrap(Server.prototype, 'handleApiRequest', wrapHandleApiRequest)
@@ -268,7 +268,7 @@ addHook({
 addHook({
   name: 'next',
   versions: ['>=10.2 <11.1'],
-  file: 'dist/next-server/server/next-server.js'
+  file: 'dist/next-server/server/next-server.js',
 }, nextServer => {
   const Server = nextServer.default
 
@@ -288,7 +288,7 @@ addHook({
 addHook({
   name: 'next',
   versions: ['>=13'],
-  file: 'dist/server/web/spec-extension/request.js'
+  file: 'dist/server/web/spec-extension/request.js',
 }, request => {
   const requestProto = Object.getPrototypeOf(request.NextRequest.prototype)