dd-trace 3.14.1 → 3.16.0

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

@@ -7,17 +7,17 @@ const shimmer = require('../../datadog-shimmer')
 const startCh = channel('apm:mariadb:query:start')
 const finishCh = channel('apm:mariadb:query:finish')
 const errorCh = channel('apm:mariadb:query:error')
+const skipCh = channel('apm:mariadb:pool:skip')
+const unskipCh = channel('apm:mariadb:pool:unskip')
 
-function wrapCommandStart (start) {
+function wrapCommandStart (start, callbackResource) {
   return function () {
     if (!startCh.hasSubscribers) return start.apply(this, arguments)
 
-    const callbackResource = new AsyncResource('bound-anonymous-fn')
-
     const resolve = callbackResource.bind(this.resolve)
     const reject = callbackResource.bind(this.reject)
 
-    const asyncResource = new AsyncResource('bound-anonymous-fn')
+    const asyncResource = callbackResource.runInAsyncScope(() => new AsyncResource('bound-anonymous-fn'))
 
     shimmer.wrap(this, 'resolve', function wrapResolve () {
       return function () {
@@ -47,17 +47,136 @@ function wrapCommandStart (start) {
   }
 }
 
+function wrapCommand (Command) {
+  return class extends Command {
+    constructor (...args) {
+      super(...args)
+
+      const callbackResource = new AsyncResource('bound-anonymous-fn')
+
+      if (this.start) {
+        this.start = wrapCommandStart(this.start, callbackResource)
+      }
+    }
+  }
+}
+
+function createWrapQuery (options) {
+  return function wrapQuery (query) {
+    return function (sql) {
+      if (!startCh.hasSubscribers) return query.apply(this, arguments)
+
+      const asyncResource = new AsyncResource('bound-anonymous-fn')
+
+      return asyncResource.runInAsyncScope(() => {
+        startCh.publish({ sql, conf: options })
+
+        return query.apply(this, arguments)
+          .then(result => {
+            finishCh.publish()
+            return result
+          }, error => {
+            errorCh.publish(error)
+            finishCh.publish()
+            throw error
+          })
+      }, 'bound-anonymous-fn')
+    }
+  }
+}
+
+function createWrapQueryCallback (options) {
+  return function wrapQuery (query) {
+    return function (sql) {
+      if (!startCh.hasSubscribers) return query.apply(this, arguments)
+
+      const cb = arguments[arguments.length - 1]
+      const asyncResource = new AsyncResource('bound-anonymous-fn')
+      const callbackResource = new AsyncResource('bound-anonymous-fn')
+
+      if (typeof cb !== 'function') {
+        arguments.length = arguments.length + 1
+      }
+
+      arguments[arguments.length - 1] = asyncResource.bind(function (err) {
+        if (err) {
+          errorCh.publish(err)
+        }
+
+        finishCh.publish()
+
+        if (typeof cb === 'function') {
+          return callbackResource.runInAsyncScope(() => cb.apply(this, arguments))
+        }
+      })
+
+      return asyncResource.runInAsyncScope(() => {
+        startCh.publish({ sql, conf: options })
+
+        return query.apply(this, arguments)
+      }, 'bound-anonymous-fn')
+    }
+  }
+}
+
+function wrapConnection (Connection, promiseMethod) {
+  return function (options) {
+    Connection.apply(this, arguments)
+
+    shimmer.wrap(this, promiseMethod, createWrapQuery(options))
+    shimmer.wrap(this, '_queryCallback', createWrapQueryCallback(options))
+  }
+}
+
+function wrapPoolBase (PoolBase) {
+  return function (options, processTask, createConnectionPool, pingPromise) {
+    arguments[1] = wrapPoolMethod(processTask)
+    arguments[2] = wrapPoolMethod(createConnectionPool)
+
+    PoolBase.apply(this, arguments)
+
+    shimmer.wrap(this, 'query', createWrapQuery(options.connOptions))
+  }
+}
+
+// It's not possible to prevent connection pools from leaking across queries,
+// so instead we just skip instrumentation completely to avoid memory leaks
+// and/or orphan spans.
+function wrapPoolMethod (createConnection) {
+  return function () {
+    skipCh.publish()
+    try {
+      return createConnection.apply(this, arguments)
+    } finally {
+      unskipCh.publish()
+    }
+  }
+}
+
 const name = 'mariadb'
-const versions = ['>=2.0.3']
 
-addHook({ name, file: 'lib/cmd/query.js', versions }, (Query) => {
-  shimmer.wrap(Query.prototype, 'start', wrapCommandStart)
+addHook({ name, file: 'lib/cmd/query.js', versions: ['>=3'] }, (Query) => {
+  return wrapCommand(Query)
+})
+
+addHook({ name, file: 'lib/cmd/execute.js', versions: ['>=3'] }, (Execute) => {
+  return wrapCommand(Execute)
+})
+
+addHook({ name, file: 'lib/pool.js', versions: ['>=3'] }, (Pool) => {
+  shimmer.wrap(Pool.prototype, '_createConnection', wrapPoolMethod)
 
-  return Query
+  return Pool
 })
 
-addHook({ name, file: 'lib/cmd/execute.js', versions }, (Query) => {
-  shimmer.wrap(Query.prototype, 'start', wrapCommandStart)
+addHook({ name, file: 'lib/connection.js', versions: ['>=2.5.2 <3'] }, (Connection) => {
+  return shimmer.wrap(Connection, wrapConnection(Connection, '_queryPromise'))
+})
+
+addHook({ name, file: 'lib/connection.js', versions: ['>=2.0.4 <=2.5.1'] }, (Connection) => {
+  return shimmer.wrap(Connection, wrapConnection(Connection, 'query'))
+})
 
-  return Query
+addHook({ name, file: 'lib/pool-base.js', versions: ['>=2.0.4 <3'] }, (PoolBase) => {
+  return shimmer.wrap(PoolBase, wrapPoolBase(PoolBase))
 })

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

@@ -2,6 +2,8 @@ const { createCoverageMap } = require('istanbul-lib-coverage')
 
 const { addHook, channel, AsyncResource } = require('./helpers/instrument')
 const shimmer = require('../../datadog-shimmer')
+const log = require('../../dd-trace/src/log')
+
 const {
   getCoveredFilenamesFromCoverage,
   resetCoverage,
@@ -34,6 +36,9 @@ const testToAr = new WeakMap()
 const originalFns = new WeakMap()
 const testFileToSuiteAr = new Map()
 
+// `isWorker` is true if it's a Mocha worker
+let isWorker = false
+
 // We'll preserve the original coverage here
 const originalCoverageMap = createCoverageMap()
 
@@ -101,7 +106,7 @@ function mochaHook (Runner) {
   patched.add(Runner)
 
   shimmer.wrap(Runner.prototype, 'run', run => function () {
-    if (!testStartCh.hasSubscribers) {
+    if (!testStartCh.hasSubscribers || isWorker) {
       return run.apply(this, arguments)
     }
 
@@ -120,9 +125,18 @@ function mochaHook (Runner) {
 
       const isSuitesSkipped = !!suitesToSkip.length
 
-      testSessionFinishCh.publish({ status, isSuitesSkipped })
-      // restore the original coverage
-      global.__coverage__ = fromCoverageMapToCoverage(originalCoverageMap)
+      let testCodeCoverageLinesTotal
+      if (global.__coverage__) {
+        try {
+          testCodeCoverageLinesTotal = originalCoverageMap.getCoverageSummary().lines.pct
+        } catch (e) {
+          // ignore errors
+        }
+        // restore the original coverage
+        global.__coverage__ = fromCoverageMapToCoverage(originalCoverageMap)
+      }
+
+      testSessionFinishCh.publish({ status, isSuitesSkipped, testCodeCoverageLinesTotal })
     }))
 
     this.once('start', testRunAsyncResource.bind(function () {
@@ -321,7 +335,15 @@ addHook({
    * If ITR is disabled, `onDone` is called immediately on the subscriber
    */
   shimmer.wrap(Mocha.prototype, 'run', run => function () {
-    if (!itrConfigurationCh.hasSubscribers) {
+    if (this.options.parallel) {
+      log.warn(`Unable to initialize CI Visibility because Mocha is running in parallel mode.`)
+      return run.apply(this, arguments)
+    }
+
+    if (!itrConfigurationCh.hasSubscribers || this.isWorker) {
+      if (this.isWorker) {
+        isWorker = true
+      }
       return run.apply(this, arguments)
     }
     this.options.delay = true
@@ -379,7 +401,9 @@ addHook({
      * This attaches `run` to the global context, which we'll call after
      * our configuration and skippable suites requests
      */
-    mocha.options.delay = true
+    if (!mocha.options.parallel) {
+      mocha.options.delay = true
+    }
     return runMocha.apply(this, arguments)
   })
   return run

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

@@ -25,13 +25,19 @@ addHook({ name: 'mongodb-core', versions: ['2 - 3.1.9'] }, Server => {
   return Server
 })
 
-addHook({ name: 'mongodb', versions: ['>=4'], file: 'lib/cmap/connection.js' }, Connection => {
+addHook({ name: 'mongodb', versions: ['>=4 <4.6.0'], file: 'lib/cmap/connection.js' }, Connection => {
   const proto = Connection.Connection.prototype
   shimmer.wrap(proto, 'command', command => wrapConnectionCommand(command, 'command'))
   shimmer.wrap(proto, 'query', query => wrapConnectionCommand(query, 'query'))
   return Connection
 })
 
+addHook({ name: 'mongodb', versions: ['>=4.6.0'], file: 'lib/cmap/connection.js' }, Connection => {
+  const proto = Connection.Connection.prototype
+  shimmer.wrap(proto, 'command', command => wrapConnectionCommand(command, 'command'))
+  return Connection
+})
+
 addHook({ name: 'mongodb', versions: ['>=3.3 <4'], file: 'lib/core/wireprotocol/index.js' }, wp => wrapWp(wp))
 
 addHook({ name: 'mongodb-core', versions: ['>=3.2'], file: 'lib/wireprotocol/index.js' }, wp => wrapWp(wp))
@@ -46,7 +52,7 @@ addHook({ name: 'mongodb-core', versions: ['~3.1.10'], file: 'lib/wireprotocol/2
   return WireProtocol
 })
 
-addHook({ name: 'mongodb', versions: ['>=3.5.4'], file: 'lib/utils.js' }, util => {
+addHook({ name: 'mongodb', versions: ['>=3.5.4 <4.11.0'], file: 'lib/utils.js' }, util => {
   shimmer.wrap(util, 'maybePromise', maybePromise => function (parent, callback, fn) {
     const asyncResource = new AsyncResource('bound-anonymous-fn')
     const callbackIndex = arguments.length - 2

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

@@ -19,7 +19,7 @@ function wrapAddQueue (addQueue) {
 
 addHook({
   name: 'mongoose',
-  versions: ['>=4.6.4']
+  versions: ['>=4.6.4 <5', '5', '6', '>=7']
 }, mongoose => {
   if (mongoose.Promise !== global.Promise) {
     shimmer.wrap(mongoose.Promise.prototype, 'then', wrapThen)

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

@@ -26,16 +26,33 @@ function wrapHandleApiRequest (handleApiRequest) {
       return promise.then(handled => {
         if (!handled) return handled
 
-        const page = getPageFromPath(pathname, this.dynamicRoutes)
+        return this.hasPage(pathname).then(pageFound => {
+          const page = pageFound ? pathname : getPageFromPath(pathname, this.dynamicRoutes)
 
-        pageLoadChannel.publish({ page })
+          pageLoadChannel.publish({ page })
 
-        return handled
+          return handled
+        })
       })
     })
   }
 }
 
+// next 13.2 handleApiRequest uses a different set of parameters
+function wrapHandleApiRequestWithMatch (handleApiRequest) {
+  return function (req, res, query, match) {
+    return instrument(req, res, () => {
+      const page = (typeof match === 'object' && typeof match.definition === 'object')
+        ? match.definition.pathname
+        : undefined
+
+      pageLoadChannel.publish({ page })
+
+      return handleApiRequest.apply(this, arguments)
+    })
+  }
+}
+
 function wrapRenderToResponse (renderToResponse) {
   return function (ctx) {
     return instrument(ctx.req, ctx.res, () => renderToResponse.apply(this, arguments))
@@ -127,7 +144,19 @@ function finish (req, res, result, err) {
   return result
 }
 
-addHook({ name: 'next', versions: ['>=11.1'], file: 'dist/server/next-server.js' }, nextServer => {
+addHook({ name: 'next', versions: ['>=13.2'], file: 'dist/server/next-server.js' }, nextServer => {
+  const Server = nextServer.default
+
+  shimmer.wrap(Server.prototype, 'handleRequest', wrapHandleRequest)
+  shimmer.wrap(Server.prototype, 'handleApiRequest', wrapHandleApiRequestWithMatch)
+  shimmer.wrap(Server.prototype, 'renderToResponse', wrapRenderToResponse)
+  shimmer.wrap(Server.prototype, 'renderErrorToResponse', wrapRenderErrorToResponse)
+  shimmer.wrap(Server.prototype, 'findPageComponents', wrapFindPageComponents)
+
+  return nextServer
+})
+
+addHook({ name: 'next', versions: ['>=11.1 <13.2'], file: 'dist/server/next-server.js' }, nextServer => {
   const Server = nextServer.default
 
   shimmer.wrap(Server.prototype, 'handleRequest', wrapHandleRequest)

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

@@ -25,6 +25,17 @@ const STATUS_TO_TEST_STATUS = {
 
 let remainingTestsByFile = {}
 
+function getTestsBySuiteFromTestGroups (testGroups) {
+  return testGroups.reduce((acc, { requireFile, tests }) => {
+    if (acc[requireFile]) {
+      acc[requireFile] = acc[requireFile].concat(tests)
+    } else {
+      acc[requireFile] = tests
+    }
+    return acc
+  }, {})
+}
+
 function getTestsBySuiteFromTestsById (testsById) {
   const testsByTestSuite = {}
   for (const { test } of testsById.values()) {
@@ -48,7 +59,7 @@ function getPlaywrightConfig (playwrightRunner) {
     try {
       return playwrightRunner._loader.fullConfig()
     } catch (e) {
-      return {}
+      return playwrightRunner._config || {}
     }
   }
 }
@@ -65,7 +76,7 @@ function getRootDir (playwrightRunner) {
 }
 
 function testBeginHandler (test) {
-  const { title: testName, location: { file: testSuiteAbsolutePath }, _type } = test
+  const { _requireFile: testSuiteAbsolutePath, title: testName, _type } = test
 
   if (_type === 'beforeAll' || _type === 'afterAll') {
     return
@@ -90,7 +101,7 @@ function testBeginHandler (test) {
 }
 
 function testEndHandler (test, testStatus, error) {
-  const { location: { file: testSuiteAbsolutePath }, results, _type } = test
+  const { _requireFile: testSuiteAbsolutePath, results, _type } = test
 
   if (_type === 'beforeAll' || _type === 'afterAll') {
     return
@@ -135,6 +146,13 @@ function dispatcherRunWrapper (run) {
   }
 }
 
+function dispatcherRunWrapperNew (run) {
+  return function () {
+    remainingTestsByFile = getTestsBySuiteFromTestGroups(arguments[0])
+    return run.apply(this, arguments)
+  }
+}
+
 function dispatcherHook (dispatcherExport) {
   shimmer.wrap(dispatcherExport.Dispatcher.prototype, 'run', dispatcherRunWrapper)
   shimmer.wrap(dispatcherExport.Dispatcher.prototype, '_createWorker', createWorker => function () {
@@ -160,8 +178,8 @@ function dispatcherHook (dispatcherExport) {
   return dispatcherExport
 }
 
-function dispatcherHookNew (dispatcherExport) {
-  shimmer.wrap(dispatcherExport.Dispatcher.prototype, 'run', dispatcherRunWrapper)
+function dispatcherHookNew (dispatcherExport, runWrapper) {
+  shimmer.wrap(dispatcherExport.Dispatcher.prototype, 'run', runWrapper)
   shimmer.wrap(dispatcherExport.Dispatcher.prototype, '_createWorker', createWorker => function () {
     const dispatcher = this
     const worker = createWorker.apply(this, arguments)
@@ -192,24 +210,23 @@ function runnerHook (runnerExport, playwrightVersion) {
       testSessionStartCh.publish({ command, frameworkVersion: playwrightVersion, rootDir })
     })
 
-    const res = await runAllTests.apply(this, arguments)
-    const sessionStatus = STATUS_TO_TEST_STATUS[res.status]
+    const runAllTestsReturn = await runAllTests.apply(this, arguments)
+    const sessionStatus = runAllTestsReturn.status || runAllTestsReturn
 
     let onDone
 
     const flushWait = new Promise(resolve => {
       onDone = resolve
     })
-
     testSessionAsyncResource.runInAsyncScope(() => {
-      testSessionFinishCh.publish({ status: sessionStatus, onDone })
+      testSessionFinishCh.publish({ status: STATUS_TO_TEST_STATUS[sessionStatus], onDone })
     })
     await flushWait
 
     startedSuites = []
     remainingTestsByFile = {}
 
-    return res
+    return runAllTestsReturn
   })
 
   return runnerExport
@@ -218,7 +235,7 @@ function runnerHook (runnerExport, playwrightVersion) {
 addHook({
   name: '@playwright/test',
   file: 'lib/runner.js',
-  versions: ['>=1.18.0']
+  versions: ['>=1.18.0 <1.30.0']
 }, runnerHook)
 
 addHook({
@@ -230,5 +247,17 @@ addHook({
 addHook({
   name: '@playwright/test',
   file: 'lib/dispatcher.js',
-  versions: ['>=1.30.0']
-}, dispatcherHookNew)
+  versions: ['>=1.30.0 <1.31.0']
+}, (dispatcher) => dispatcherHookNew(dispatcher, dispatcherRunWrapper))
+
+addHook({
+  name: '@playwright/test',
+  file: 'lib/runner/dispatcher.js',
+  versions: ['>=1.31.0']
+}, (dispatcher) => dispatcherHookNew(dispatcher, dispatcherRunWrapperNew))
+
+addHook({
+  name: '@playwright/test',
+  file: 'lib/runner/runner.js',
+  versions: ['>=1.31.0']
+}, runnerHook)

package/packages/datadog-plugin-amqp10/src/consumer.js

@@ -4,7 +4,7 @@ const ConsumerPlugin = require('../../dd-trace/src/plugins/consumer')
 const { getAddress, getShortName } = require('./util')
 
 class Amqp10ConsumerPlugin extends ConsumerPlugin {
-  static get name () { return 'amqp10' }
+  static get id () { return 'amqp10' }
   static get system () { return 'amqp' }
 
   start ({ link }) {

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

@@ -5,7 +5,7 @@ const ConsumerPlugin = require('./consumer')
 const CompositePlugin = require('../../dd-trace/src/plugins/composite')
 
 class Amqp10Plugin extends CompositePlugin {
-  static get name () { return 'amqp10' }
+  static get id () { return 'amqp10' }
   static get plugins () {
     return {
       producer: ProducerPlugin,

package/packages/datadog-plugin-amqp10/src/producer.js

@@ -1,10 +1,11 @@
 'use strict'
 
 const ProducerPlugin = require('../../dd-trace/src/plugins/producer')
+const { CLIENT_PORT_KEY } = require('../../dd-trace/src/constants')
 const { getAddress, getShortName } = require('./util')
 
 class Amqp10ProducerPlugin extends ProducerPlugin {
-  static get name () { return 'amqp10' }
+  static get id () { return 'amqp10' }
   static get operation () { return 'send' }
   static get system () { return 'amqp' }
 
@@ -20,7 +21,7 @@ class Amqp10ProducerPlugin extends ProducerPlugin {
         'amqp.link.target.address': target,
         'amqp.link.role': 'sender',
         'out.host': address.host,
-        'out.port': address.port,
+        [CLIENT_PORT_KEY]: address.port,
         'amqp.link.name': link.name,
         'amqp.link.handle': link.handle,
         'amqp.connection.host': address.host,

package/packages/datadog-plugin-amqplib/src/client.js

@@ -1,11 +1,12 @@
 'use strict'
 
 const { TEXT_MAP } = require('../../../ext/formats')
+const { CLIENT_PORT_KEY } = require('../../dd-trace/src/constants')
 const ClientPlugin = require('../../dd-trace/src/plugins/client')
 const { getResourceName } = require('./util')
 
 class AmqplibClientPlugin extends ClientPlugin {
-  static get name () { return 'amqplib' }
+  static get id () { return 'amqplib' }
   static get operation () { return 'command' }
 
   start ({ channel = {}, method, fields }) {
@@ -19,7 +20,7 @@ class AmqplibClientPlugin extends ClientPlugin {
       kind: 'client',
       meta: {
         'out.host': stream._host,
-        'out.port': stream.remotePort,
+        [CLIENT_PORT_KEY]: stream.remotePort,
         'amqp.queue': fields.queue,
         'amqp.exchange': fields.exchange,
         'amqp.routingKey': fields.routingKey,

package/packages/datadog-plugin-amqplib/src/consumer.js

@@ -5,7 +5,7 @@ const ConsumerPlugin = require('../../dd-trace/src/plugins/consumer')
 const { getResourceName } = require('./util')
 
 class AmqplibConsumerPlugin extends ConsumerPlugin {
-  static get name () { return 'amqplib' }
+  static get id () { return 'amqplib' }
   static get operation () { return 'command' }
 
   start ({ method, fields, message }) {

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

@@ -7,7 +7,7 @@ const CompositePlugin = require('../../dd-trace/src/plugins/composite')
 
 // TODO: Consider splitting channels for publish/receive in the instrumentation.
 class AmqplibPlugin extends CompositePlugin {
-  static get name () { return 'amqplib' }
+  static get id () { return 'amqplib' }
   static get plugins () {
     return {
       producer: ProducerPlugin,

package/packages/datadog-plugin-amqplib/src/producer.js

@@ -1,11 +1,12 @@
 'use strict'
 
 const { TEXT_MAP } = require('../../../ext/formats')
+const { CLIENT_PORT_KEY } = require('../../dd-trace/src/constants')
 const ProducerPlugin = require('../../dd-trace/src/plugins/producer')
 const { getResourceName } = require('./util')
 
 class AmqplibProducerPlugin extends ProducerPlugin {
-  static get name () { return 'amqplib' }
+  static get id () { return 'amqplib' }
   static get operation () { return 'command' }
 
   start ({ channel = {}, method, fields }) {
@@ -18,7 +19,7 @@ class AmqplibProducerPlugin extends ProducerPlugin {
       kind: 'producer',
       meta: {
         'out.host': stream._host,
-        'out.port': stream.remotePort,
+        [CLIENT_PORT_KEY]: stream.remotePort,
         'amqp.queue': fields.queue,
         'amqp.exchange': fields.exchange,
         'amqp.routingKey': fields.routingKey,

package/packages/datadog-plugin-aws-sdk/src/base.js

@@ -3,10 +3,13 @@
 const analyticsSampler = require('../../dd-trace/src/analytics_sampler')
 const Plugin = require('../../dd-trace/src/plugins/plugin')
 const { storage } = require('../../datadog-core')
+const { isTrue } = require('../../dd-trace/src/util')
 
 class BaseAwsSdkPlugin extends Plugin {
+  static get id () { return 'aws' }
+
   get serviceIdentifier () {
-    const id = this.constructor.name.toLowerCase()
+    const id = this.constructor.id.toLowerCase()
     Object.defineProperty(this, 'serviceIdentifier', {
       configurable: true,
       writable: true,
@@ -74,7 +77,9 @@ class BaseAwsSdkPlugin extends Plugin {
   }
 
   isEnabled (request) {
-    return true
+    const serviceId = this.serviceIdentifier.toUpperCase()
+    const envVarValue = process.env[`DD_TRACE_AWS_SDK_${serviceId}_ENABLED`]
+    return envVarValue ? isTrue(envVarValue) : true
   }
 
   addResponseTags (span, response) {

package/packages/datadog-plugin-aws-sdk/src/index.js

@@ -5,7 +5,7 @@ const Plugin = require('../../dd-trace/src/plugins/plugin')
 const services = require('./services')
 
 class AwsSdkPlugin extends Plugin {
-  static get name () {
+  static get id () {
     return 'aws-sdk'
   }
 

package/packages/datadog-plugin-aws-sdk/src/services/cloudwatchlogs.js

@@ -3,6 +3,8 @@
 const BaseAwsSdkPlugin = require('../base')
 
 class CloudwatchLogs extends BaseAwsSdkPlugin {
+  static get id () { return 'cloudwatchlogs' }
+
   generateTags (params, operation) {
     const tags = {}
 

package/packages/datadog-plugin-aws-sdk/src/services/dynamodb.js

@@ -3,6 +3,8 @@
 const BaseAwsSdkPlugin = require('../base')
 
 class DynamoDb extends BaseAwsSdkPlugin {
+  static get id () { return 'dynamodb' }
+
   generateTags (params, operation, response) {
     const tags = {}
 

package/packages/datadog-plugin-aws-sdk/src/services/eventbridge.js

@@ -3,6 +3,8 @@ const log = require('../../../dd-trace/src/log')
 const BaseAwsSdkPlugin = require('../base')
 
 class EventBridge extends BaseAwsSdkPlugin {
+  static get id () { return 'eventbridge' }
+
   generateTags (params, operation, response) {
     if (!params || !params.source) return {}
 

package/packages/datadog-plugin-aws-sdk/src/services/kinesis.js

@@ -2,6 +2,8 @@
 const log = require('../../../dd-trace/src/log')
 const BaseAwsSdkPlugin = require('../base')
 class Kinesis extends BaseAwsSdkPlugin {
+  static get id () { return 'kinesis' }
+
   generateTags (params, operation, response) {
     if (!params || !params.StreamName) return {}
 

package/packages/datadog-plugin-aws-sdk/src/services/lambda.js

@@ -4,6 +4,8 @@ const log = require('../../../dd-trace/src/log')
 const BaseAwsSdkPlugin = require('../base')
 
 class Lambda extends BaseAwsSdkPlugin {
+  static get id () { return 'lambda' }
+
   generateTags (params, operation, response) {
     const tags = {}
 

package/packages/datadog-plugin-aws-sdk/src/services/redshift.js

@@ -3,6 +3,8 @@
 const BaseAwsSdkPlugin = require('../base')
 
 class Redshift extends BaseAwsSdkPlugin {
+  static get id () { return 'redshift' }
+
   generateTags (params, operation, response) {
     const tags = {}