newrelic 12.3.1 → 12.5.0

package/NEWS.md

@@ -1,24 +1,67 @@
-### v12.3.1 (2024-09-04)
-
-#### Bug fixes
-
-* Fixed detection of REST API type payloads in AWS Lambda ([#2543](https://github.com/newrelic/node-newrelic/pull/2543)) ([adfeebc](https://github.com/newrelic/node-newrelic/commit/adfeebc043161e0e0c35de2cf93989dbde9cb8fa))
-
-#### Documentation
-
-* Cleaned up formatting of api.js to properly inject example snippets when rendering on API docs site ([#2524](https://github.com/newrelic/node-newrelic/pull/2524)) ([4b34f3d](https://github.com/newrelic/node-newrelic/commit/4b34f3dbab45a55ec447b6e21b69c7621b41e539))
-* Updated compatibility report ([#2523](https://github.com/newrelic/node-newrelic/pull/2523)) ([29784ea](https://github.com/newrelic/node-newrelic/commit/29784ea766b2a9388c050f271ab7190895bc22ed))
-* Updated Next.js Otel cloud provider FAQ ([#2537](https://github.com/newrelic/node-newrelic/pull/2537)) ([6553807](https://github.com/newrelic/node-newrelic/commit/655380760a89193c5b6cd47d3955d1244cd79e7b))
-
-#### Tests
-
-* Converted db unit tests to node:test ([#2514](https://github.com/newrelic/node-newrelic/pull/2514)) ([bea4548](https://github.com/newrelic/node-newrelic/commit/bea45481a8a04099096929b36532203fbb8b6921))
-* Converted grpc, lib, and utilization tests to `node:test` ([#2532](https://github.com/newrelic/node-newrelic/pull/2532)) ([c207e1e](https://github.com/newrelic/node-newrelic/commit/c207e1e3de75a9c3a2c4a05fa1bc318d3e455ef9))
-* Replaced distributed tracing tests with `node:test` ([#2527](https://github.com/newrelic/node-newrelic/pull/2527)) ([8184c56](https://github.com/newrelic/node-newrelic/commit/8184c5676155b9028c84adc0da3902803ee9d107))
-* Added a match function for tests ([#2541](https://github.com/newrelic/node-newrelic/pull/2541)) ([51e7f34](https://github.com/newrelic/node-newrelic/commit/51e7f34e733202a9c2c024d9d9a7f3c207dfc4b0))
-* Converted `config` to `node:test` ([#2517](https://github.com/newrelic/node-newrelic/pull/2517)) ([1534a73](https://github.com/newrelic/node-newrelic/commit/1534a734995b6800c4cab3b6712f1b6b1329ed5e))
-
-
+### v12.5.0 (2024-09-12)
+
+#### Features
+
+* Added utilization info for ECS ([#2565](https://github.com/newrelic/node-newrelic/pull/2565)) ([6f92073](https://github.com/newrelic/node-newrelic/commit/6f92073a6c01124d8ab1b54d06c176a36fbc3441))
+
+#### Bug fixes
+
+* Ensured README displays for Azure site extension ([#2564](https://github.com/newrelic/node-newrelic/pull/2564)) ([a30aed5](https://github.com/newrelic/node-newrelic/commit/a30aed5cf31c0c89678618e51215063562331848))
+
+#### Documentation
+
+* Updated compatibility report ([#2562](https://github.com/newrelic/node-newrelic/pull/2562)) ([8f7aebe](https://github.com/newrelic/node-newrelic/commit/8f7aebe7e4274ce45cfe961537a09b34077b3aa0))
+
+#### Tests
+
+* Convert `metric` and `metrics-recorder` tests to `node:test` ([#2552](https://github.com/newrelic/node-newrelic/pull/2552)) ([7ae4af4](https://github.com/newrelic/node-newrelic/commit/7ae4af4c8adfabadd3c865bd2fdd0e8ba5317eef))
+* Updated `serverless` unit tests to `node:test` ([#2549](https://github.com/newrelic/node-newrelic/pull/2549)) ([619f23c](https://github.com/newrelic/node-newrelic/commit/619f23c938bf39c360a6da9a307c178986c70902))
+
+### v12.4.0 (2024-09-11)
+
+#### Features
+
+* Added support for `express@5` ([#2555](https://github.com/newrelic/node-newrelic/pull/2555)) ([252f3b2](https://github.com/newrelic/node-newrelic/commit/252f3b2bc1206dad52d914b98a2352da317da2d5))
+* Provided ability to disable instrumentation for a 3rd party package ([#2551](https://github.com/newrelic/node-newrelic/pull/2551)) ([abfb9f0](https://github.com/newrelic/node-newrelic/commit/abfb9f029a4f6c25966c35d3284ddae0d46dfecb))
+  * To disable instrumentation set `config.instrumentation.<library>.enabled` to false.  The values of `<library>` are the keys listed [here](https://github.com/newrelic/node-newrelic/blob/main/lib/instrumentations.js)
+  * This feature is use at your own risk.  Disabling instrumentation for a library could affect instrumentation of other libraries executed afterwards.  
+  
+
+#### Miscellaneous chores
+
+* Added CI for publishing agent as Azure site extension ([#2488](https://github.com/newrelic/node-newrelic/pull/2488)) ([468943a](https://github.com/newrelic/node-newrelic/commit/468943a1ed3864dafb93a2f96561d1a778d03a5f))
+* Added Azure site extension installation scripts ([#2448](https://github.com/newrelic/node-newrelic/pull/2448)) ([a56c4e1](https://github.com/newrelic/node-newrelic/commit/a56c4e146ead7d3205fead1f17afad0ea7a77e59))
+
+#### Tests
+
+* Converted `llm-events` tests to use `node:test` ([#2535](https://github.com/newrelic/node-newrelic/pull/2535)) ([ebfa2e9](https://github.com/newrelic/node-newrelic/commit/ebfa2e9ab8ecbe4bc9adaddd3e4a60e3ba84d0d9))
+* Migrated `test/unit/spans` to use `node:test` ([#2556](https://github.com/newrelic/node-newrelic/pull/2556)) ([9319071](https://github.com/newrelic/node-newrelic/commit/931907182b0168990a04bb92c2f28310450f8ba0))
+* Migrated `test/unit/util` to use `node:test` ([#2546](https://github.com/newrelic/node-newrelic/pull/2546)) ([0b07be8](https://github.com/newrelic/node-newrelic/commit/0b07be8f7f29e67630326c73b96faa5e20527a0b))
+* Migrated tests in `test/unit/instrumentation` to use `node:test` ([#2531](https://github.com/newrelic/node-newrelic/pull/2531)) ([47b8398](https://github.com/newrelic/node-newrelic/commit/47b8398820d665a85a96ae84e30eaaf20564dcf8))
+* Converted `collector` unit tests to `node:test` ([#2510](https://github.com/newrelic/node-newrelic/pull/2510)) ([762511b](https://github.com/newrelic/node-newrelic/commit/762511be524f971a609ff45c111c2d1a89ec1c46))
+* Converted `errors` unit tests to `node:test` ([#2540](https://github.com/newrelic/node-newrelic/pull/2540)) ([ae82760](https://github.com/newrelic/node-newrelic/commit/ae82760f7001f6bcdd6a9fe0ec1e96dc60db99e5))
+
+### v12.3.1 (2024-09-04)
+
+#### Bug fixes
+
+* Fixed detection of REST API type payloads in AWS Lambda ([#2543](https://github.com/newrelic/node-newrelic/pull/2543)) ([adfeebc](https://github.com/newrelic/node-newrelic/commit/adfeebc043161e0e0c35de2cf93989dbde9cb8fa))
+
+#### Documentation
+
+* Cleaned up formatting of api.js to properly inject example snippets when rendering on API docs site ([#2524](https://github.com/newrelic/node-newrelic/pull/2524)) ([4b34f3d](https://github.com/newrelic/node-newrelic/commit/4b34f3dbab45a55ec447b6e21b69c7621b41e539))
+* Updated compatibility report ([#2523](https://github.com/newrelic/node-newrelic/pull/2523)) ([29784ea](https://github.com/newrelic/node-newrelic/commit/29784ea766b2a9388c050f271ab7190895bc22ed))
+* Updated Next.js Otel cloud provider FAQ ([#2537](https://github.com/newrelic/node-newrelic/pull/2537)) ([6553807](https://github.com/newrelic/node-newrelic/commit/655380760a89193c5b6cd47d3955d1244cd79e7b))
+
+#### Tests
+
+* Converted db unit tests to node:test ([#2514](https://github.com/newrelic/node-newrelic/pull/2514)) ([bea4548](https://github.com/newrelic/node-newrelic/commit/bea45481a8a04099096929b36532203fbb8b6921))
+* Converted grpc, lib, and utilization tests to `node:test` ([#2532](https://github.com/newrelic/node-newrelic/pull/2532)) ([c207e1e](https://github.com/newrelic/node-newrelic/commit/c207e1e3de75a9c3a2c4a05fa1bc318d3e455ef9))
+* Replaced distributed tracing tests with `node:test` ([#2527](https://github.com/newrelic/node-newrelic/pull/2527)) ([8184c56](https://github.com/newrelic/node-newrelic/commit/8184c5676155b9028c84adc0da3902803ee9d107))
+* Added a match function for tests ([#2541](https://github.com/newrelic/node-newrelic/pull/2541)) ([51e7f34](https://github.com/newrelic/node-newrelic/commit/51e7f34e733202a9c2c024d9d9a7f3c207dfc4b0))
+* Converted `config` to `node:test` ([#2517](https://github.com/newrelic/node-newrelic/pull/2517)) ([1534a73](https://github.com/newrelic/node-newrelic/commit/1534a734995b6800c4cab3b6712f1b6b1329ed5e))
+
+
 ### v12.3.0 (2024-08-27)
 
 #### Features

package/THIRD_PARTY_NOTICES.md

@@ -34,6 +34,7 @@ code, the source code can be found at [https://github.com/newrelic/node-newrelic
 * [@aws-sdk/client-s3](#aws-sdkclient-s3)
 * [@aws-sdk/s3-request-presigner](#aws-sdks3-request-presigner)
 * [@koa/router](#koarouter)
+* [@matteo.collina/tspl](#matteocollinatspl)
 * [@newrelic/eslint-config](#newreliceslint-config)
 * [@newrelic/newrelic-oss-cli](#newrelicnewrelic-oss-cli)
 * [@newrelic/test-utilities](#newrelictest-utilities)
@@ -69,7 +70,6 @@ code, the source code can be found at [https://github.com/newrelic/node-newrelic
 * [nock](#nock)
 * [proxy](#proxy)
 * [proxyquire](#proxyquire)
-* [rfdc](#rfdc)
 * [rimraf](#rimraf)
 * [self-cert](#self-cert)
 * [should](#should)
@@ -1521,6 +1521,35 @@ THE SOFTWARE.
 
 ```
 
+### @matteo.collina/tspl
+
+This product includes source derived from [@matteo.collina/tspl](https://github.com/mcollina/tspl) ([v0.1.1](https://github.com/mcollina/tspl/tree/v0.1.1)), distributed under the [MIT License](https://github.com/mcollina/tspl/blob/v0.1.1/LICENSE):
+
+```
+MIT License
+
+Copyright (c) 2023 Matteo Collina
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+
+```
+
 ### @newrelic/eslint-config
 
 This product includes source derived from [@newrelic/eslint-config](https://github.com/newrelic/eslint-config-newrelic) ([v0.3.0](https://github.com/newrelic/eslint-config-newrelic/tree/v0.3.0)), distributed under the [Apache-2.0 License](https://github.com/newrelic/eslint-config-newrelic/blob/v0.3.0/LICENSE):
@@ -3978,29 +4007,6 @@ OTHER DEALINGS IN THE SOFTWARE.
 
 ```
 
-### rfdc
-
-This product includes source derived from [rfdc](https://github.com/davidmarkclements/rfdc) ([v1.4.1](https://github.com/davidmarkclements/rfdc/tree/v1.4.1)), distributed under the [MIT License](https://github.com/davidmarkclements/rfdc/blob/v1.4.1/LICENSE):
-
-```
-Copyright 2019 "David Mark Clements <david.mark.clements@gmail.com>"
-
-Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated 
-documentation files (the "Software"), to deal in the Software without restriction, including without limitation 
-the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and 
-to permit persons to whom the Software is furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all copies or substantial portions 
-of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED 
-TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL 
-THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF 
-CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS 
-IN THE SOFTWARE.
-
-```
-
 ### rimraf
 
 This product includes source derived from [rimraf](https://github.com/isaacs/rimraf) ([v2.7.1](https://github.com/isaacs/rimraf/tree/v2.7.1)), distributed under the [ISC License](https://github.com/isaacs/rimraf/blob/v2.7.1/LICENSE):

package/lib/agent.js

@@ -57,6 +57,94 @@ const MAX_ERROR_TRACES_DEFAULT = 20
 const INITIAL_HARVEST_DELAY_MS = 1000
 const DEFAULT_HARVEST_INTERVAL_MS = 60000
 
+/**
+ * Indicates that the agent has finished connecting. It is preceded by
+ * {@link Agent#event:connecting}.
+ *
+ * @event Agent#connected
+ */
+
+/**
+ * Indicates that the agent is in the connecting state. That is, it is
+ * communicating with the New Relic data collector and performing requisite
+ * operations before the agent is ready to collect and send data.
+ *
+ * @event Agent#connecting
+ */
+
+/**
+ * Indicates that the agent has terminated, or lost, its connection to the
+ * New Relic data collector.
+ *
+ * @event Agent#disconnected
+ */
+
+/**
+ * Indicates that the agent has encountered, or generated, some error that
+ * prevents it from operating correctly. The agent state will be set to
+ * "errored."
+ *
+ * @event Agent#errored
+ */
+
+/**
+ * Indicates that the synchronous harvest cycle has completed. This will be
+ * prefaced by the {@link Agent#event:harvestStarted} event. It is only fired
+ * in a serverless context.
+ *
+ * @event Agent#harvestFinished
+ */
+
+/**
+ * Indicates that a synchronous harvest cycle (collecting data from the various
+ * aggregators and sending said data to the New Relic data collector) has
+ * started. This is only fired in a serverless context.
+ *
+ * @event Agent#harvestStarted
+ */
+
+/**
+ * Indicates that the agent state has entered the "started" state. That is,
+ * the agent has finished bootstrapping and is collecting and sending data.
+ *
+ * @event Agent#started
+ */
+
+/**
+ * Indicates that the agent is starting. This is typically the first event
+ * emitted by the agent.
+ *
+ * @event Agent#starting
+ */
+
+/**
+ * Indicates that the agent state has changed to the "stopped" state. That is,
+ * the agent is no longer collecting or sending data.
+ *
+ * @event Agent#stopped
+ */
+
+/**
+ * Indicates that the agent is entering its shutdown process.
+ *
+ * @event Agent#stopping
+ */
+
+/**
+ * Indicates that the transaction has begun recording data.
+ *
+ * @event Agent#transactionStarted
+ * @param {Transaction} currentTransaction
+ */
+
+/**
+ * Indicates that the transaction has stopped recording data and has been
+ * closed.
+ *
+ * @event Agent#transactionFinished
+ * @param {Transaction} currentTransaction
+ */
+
 /**
  * There's a lot of stuff in this constructor, due to Agent acting as the
  * orchestrator for New Relic within instrumented applications.
@@ -101,7 +189,7 @@ function Agent(config) {
     this.harvester
   )
 
-  this.metrics.on('starting metric_data data send.', this._beforeMetricDataSend.bind(this))
+  this.metrics.on('starting_data_send-metric_data', this._beforeMetricDataSend.bind(this))
 
   this.spanEventAggregator = createSpanEventAggregator(config, this)
 
@@ -226,6 +314,12 @@ util.inherits(Agent, EventEmitter)
  *
  * @param {Function} callback Continuation and error handler.
  * @returns {void}
+ *
+ * @fires Agent#errored When configuration is not sufficient for operations or
+ * cannot establish a connection to the data collector.
+ * @fires Agent#starting
+ * @fires Agent#stopped When configuration indicates that the agent should be
+ * disabled.
  */
 Agent.prototype.start = function start(callback) {
   if (!callback) {
@@ -306,6 +400,8 @@ Agent.prototype.startStreaming = function startStreaming() {
  *
  * @param {boolean} shouldImmediatelyHarvest Whether we should immediately schedule a harvest, or wait a cycle
  * @param {Function} callback callback function that executes after harvest completes (now if immediate, otherwise later)
+ *
+ * @fires Agent#started
  */
 Agent.prototype.onConnect = function onConnect(shouldImmediatelyHarvest, callback) {
   this.harvester.update(this.config)
@@ -368,6 +464,11 @@ Agent.prototype._serverlessModeStart = function _serverlessModeStart(callback) {
  * current instrumentation and patch to the module loader.
  *
  * @param {Function} callback callback function to invoke after agent stop
+ *
+ * @fires Agent#errored When disconnecting from the data collector has resulted
+ * in some error.
+ * @fires Agent#stopped
+ * @fires Agent#stopping
  */
 Agent.prototype.stop = function stop(callback) {
   if (!callback) {
@@ -432,9 +533,13 @@ Agent.prototype._resetCustomEvents = function resetCustomEvents() {
 }
 
 /**
- * This method invokes a harvest synchronously.
+ * This method invokes a harvest synchronously, i.e. sends all data to the
+ * New Relic collector in a blocking fashion.
+ *
+ * NOTE: this doesn't currently work outside serverless mode.
  *
- * NOTE: this doesn't currently work outside of serverless mode.
+ * @fires Agent#harvestStarted
+ * @fires Agent#harvestFinished
  */
 Agent.prototype.harvestSync = function harvestSync() {
   logger.trace('Peparing to harvest.')
@@ -518,6 +623,15 @@ Agent.prototype.reconfigure = function reconfigure(configuration) {
  * creation of Transactions.
  *
  * @param {string} newState The new state of the agent.
+ *
+ * @fires Agent#connected
+ * @fires Agent#connecting
+ * @fires Agent#disconnected
+ * @fires Agent#errored
+ * @fires Agent#started
+ * @fires Agent#starting
+ * @fires Agent#stopped
+ * @fires Agent#stopping
  */
 Agent.prototype.setState = function setState(newState) {
   if (!STATES.hasOwnProperty(newState)) {

package/lib/aggregators/base-aggregator.js

@@ -8,6 +8,152 @@
 const EventEmitter = require('events').EventEmitter
 const logger = require('../logger').child({ component: 'base_aggregator' })
 
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `analytic_event_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-analytic_event_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `analytic_event_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-analytic_event_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `custom_event_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-custom_event_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `custom_event_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-custom_event_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `error_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-error_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `error_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-error_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `error_event_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-error_event_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `error_event_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-error_event_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `log_event_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-log_event_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `log_event_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-log_event_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `metric_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-metric_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `metric_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-metric_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `span_event_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-span_event_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `span_event_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-span_event_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `sql_trace_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-sql_trace_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `sql_trace_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-sql_trace_data
+ */
+
+/**
+ * Triggered when the aggregator has finished sending data to the
+ * `transaction_sample_data` collector endpoint.
+ *
+ * @event Aggregator#finished_data_send-transaction_sample_data
+ */
+
+/**
+ * Triggered when an aggregator is sending data to the `transaction_sample_data`
+ * collector endpoint.
+ *
+ * @event Aggregator#starting_data_send-transaction_sample_data
+ */
+
+/**
+ * Baseline data aggregator that is used to ship data to the New Relic
+ * data collector. Specific data aggregators, e.g. an errors aggregator,
+ * extend this base object.
+ *
+ * Aggregators fire of several events. The events are named according to the
+ * pattern `<finished|starting>_data_send-<endpoint_name>`. As an example,
+ * if the aggregator is collecting data to send to the `error_event_data`
+ * endpoint, there will be two events:
+ *
+ * + `starting_data_send-error_event_data`
+ * + `finished_data_send-error_event_data`
+ *
+ * For a list of possible endpoints, see
+ * {@link https://source.datanerd.us/agents/agent-specs/tree/main/endpoints/protocol-version-17}.
+ *
+ * Note: effort has been made to document the events for every endpoint this
+ * agent interacts with, but due to the dynamic nature of the event names we
+ * may have missed some.
+ */
 class Aggregator extends EventEmitter {
   constructor(opts, collector, harvester) {
     super()
@@ -23,6 +169,16 @@ class Aggregator extends EventEmitter {
         return true
       }
     this.enabled = this.isEnabled(opts.config)
+
+    /**
+     * The name of the collector endpoint that the
+     * aggregator will communicate with.
+     *
+     * @see https://source.datanerd.us/agents/agent-specs/tree/main/endpoints/protocol-version-17
+     *
+     * @type {string}
+     * @memberof Aggregator
+     */
     this.method = opts.method
     this.collector = collector
     this.sendTimer = null
@@ -90,7 +246,7 @@ class Aggregator extends EventEmitter {
   _runSend(data, payload) {
     if (!payload) {
       this._afterSend(false)
-      this.emit(`finished ${this.method} data send.`)
+      this.emit(`finished_data_send-${this.method}`)
       return
     }
 
@@ -102,13 +258,36 @@ class Aggregator extends EventEmitter {
 
       // TODO: Log?
       this._afterSend(true)
-      this.emit(`finished ${this.method} data send.`)
+      this.emit(`finished_data_send-${this.method}`)
     })
   }
 
+  /**
+   * Serialize all collected data and ship it off to the New Relic data
+   * collector. The target endpoint is defined by {@link Aggregator#method}.
+   *
+   * @fires Aggregator#finished_data_send-analytic_event_data
+   * @fires Aggregator#starting_data_send-analytic_event_data
+   * @fires Aggregator#finished_data_send-custom_event_data
+   * @fires Aggregator#starting_data_send-custom_event_data
+   * @fires Aggregator#finished_data_send-error_data
+   * @fires Aggregator#starting_data_send-error_data
+   * @fires Aggregator#finished_data_send-error_event_data
+   * @fires Aggregator#starting_data_send-error_event_data
+   * @fires Aggregator#finished_data_send-log_event_data
+   * @fires Aggregator#starting_data_send-log_event_data
+   * @fires Aggregator#finished_data_send-metric_data
+   * @fires Aggregator#starting_data_send-metric_data
+   * @fires Aggregator#finished_data_send-span_event_data
+   * @fires Aggregator#starting_data_send-span_event_data
+   * @fires Aggregator#finished_data_send-sql_trace_data
+   * @fires Aggregator#starting_data_send-sql_trace_data
+   * @fires Aggregator#finished_data_send-transaction_sample_data
+   * @fires Aggregator#starting_data_send-transaction_sample_data
+   */
   send() {
     logger.debug(`${this.method} Aggregator data send.`)
-    this.emit(`starting ${this.method} data send.`)
+    this.emit(`starting_data_send-${this.method}`)
 
     const data = this._getMergeData()
     if (this.isAsync) {

package/lib/collector/api.js

@@ -30,9 +30,29 @@ const BACKOFFS = [
 
 // Expected collector response codes
 const SUCCESS = new Set([200, 202])
-const RESTART = new Set([401, 409])
-const FAILURE_SAVE_DATA = new Set([408, 429, 500, 503])
-const FAILURE_DISCARD_DATA = new Set([400, 403, 404, 405, 407, 411, 413, 414, 415, 417, 431])
+const RESTART = new Set([
+  401, // Authentication failed.
+  409 // NR says to reconnect for some reason.
+])
+const FAILURE_SAVE_DATA = new Set([
+  408, // Data took too long to reach NR.
+  429, // Too many requests being received by NR, rate limited.
+  500, // NR server went boom.
+  503 // NR server is not available.
+])
+const FAILURE_DISCARD_DATA = new Set([
+  400, // Format of the request is incorrect.
+  403, // Not entitled to perform the action.
+  404, // Sending to wrong destination.
+  405, // Using the wrong HTTP method (e.g. PUT instead of POST).
+  407, // Proxy authentication misconfigured.
+  411, // No Content-Length header provided, or value is incorrect.
+  413, // Payload is too large.
+  414, // URI exceeds allowed length.
+  415, // Content-type or Content-encoding values are incorrect.
+  417, // NR cannot meet the expectation of the request.
+  431 // Request headers exceed size limit.
+])
 
 const AGENT_RUN_BEHAVIOR = CollectorResponse.AGENT_RUN_BEHAVIOR
 
@@ -130,6 +150,17 @@ CollectorAPI.prototype._updateEndpoints = function _updateEndpoints(endpoint) {
   }
 }
 
+/**
+ * Connect to the data collector.
+ *
+ * @param {function} callback A typical error first callback to be invoked
+ * upon successful or unsuccessful connection. The second parameter will be
+ * an instance of {@link CollectorResponse}.
+ *
+ * @fires Agent#connected By way of the full connection process. This event
+ * is not fired directly in this method.
+ * @fires Agent#connecting
+ */
 CollectorAPI.prototype.connect = function connect(callback) {
   if (!callback) {
     this._throwCallbackError()
@@ -347,6 +378,8 @@ CollectorAPI.prototype._connect = function _connect(env, callback) {
  * @param {Function} callback function to run after processing response
  * @param {Error} error collector response error
  * @param {http.ServerOptions} res collector response
+ *
+ * @fires Agent#connected
  */
 CollectorAPI.prototype._onConnect = function _onConnect(callback, error, res) {
   const agent = this._agent
@@ -431,6 +464,8 @@ CollectorAPI.prototype.shutdown = function shutdown(callback) {
   /**
    * @param {Error} error response error
    * @param {http.ServerResponse} response response from collector
+   *
+   * @fires Agent#disconnected
    */
   function onShutdown(error, response) {
     if (error) {

package/lib/config/build-instrumentation-config.js

@@ -0,0 +1,19 @@
+/*
+ * Copyright 2024 New Relic Corporation. All rights reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+'use strict'
+const { boolean } = require('./formatters')
+const instrumentedLibraries = require('../instrumentations')()
+const pkgNames = Object.keys(instrumentedLibraries)
+
+/**
+ * Builds the stanza for config.instrumentation.*
+ * It defaults every library to true and assigns a boolean
+ * formatter for the environment variable conversion of the values
+ */
+module.exports = pkgNames.reduce((config, pkg) => {
+  config[pkg] = { enabled: { formatter: boolean, default: true } }
+  return config
+}, {})

package/lib/config/default.js

@@ -7,6 +7,7 @@
 
 const defaultConfig = module.exports
 const { array, int, float, boolean, object, objectList, allowList, regex } = require('./formatters')
+const pkgInstrumentation = require('./build-instrumentation-config')
 
 /**
  * A function that returns the definition of the agent configuration
@@ -1382,7 +1383,13 @@ defaultConfig.definition = () => ({
         default: true
       }
     }
-  }
+  },
+  /**
+   * Stanza that contains all keys to disable 3rd party package instrumentation(i.e. mongodb, pg, redis, etc)
+   * Note: Disabling a given 3rd party library may affect the instrumentation of 3rd party libraries used after
+   * the disabled library.
+   */
+  instrumentation: pkgInstrumentation
 })
 
 /**

package/lib/config/index.js

@@ -70,6 +70,32 @@ function _findConfigFile() {
   return configFileCandidates.find(exists)
 }
 
+/**
+ * Indicates if the agent should be enabled or disabled based upon the
+ * processed configuration.
+ *
+ * @event Config#agent_enabled
+ * @param {boolean} indication
+ */
+
+/**
+ * Indicates that the configuration has changed due to some external factor,
+ * e.g. the configuration from the New Relic server has overridden some
+ * local setting.
+ *
+ * @event Config#change
+ * @param {Config} currentConfig
+ */
+
+/**
+ * Parses and manages the agent configuration.
+ *
+ * @param {object} config Configuration object as read from a `newrelic.js`
+ * file. This is used as the baseline, which is the overridden by environment
+ * variables and feature flags.
+ *
+ * @class
+ */
 function Config(config) {
   EventEmitter.call(this)
 
@@ -194,6 +220,10 @@ Config.prototype.mergeServerConfig = mergeServerConfig
  *
  * @param {object} json The config blob sent by New Relic.
  * @param {boolean} recursion flag indicating coming from server side config
+ *
+ * @fires Config#agent_enabled When there is a conflict between the local
+ * setting of high security mode and what the New Relic server has returned.
+ * @fires Config#change
  */
 Config.prototype.onConnect = function onConnect(json, recursion) {
   json = json || Object.create(null)
@@ -274,6 +304,8 @@ Config.prototype._updateHarvestConfig = function _updateHarvestConfig(serverConf
  *
  * @param {object} params A configuration dictionary.
  * @param {string} key    The particular configuration parameter to set.
+ *
+ * @fires Config#change
  */
 Config.prototype._fromServer = function _fromServer(params, key) {
   /* eslint-disable-next-line sonarjs/max-switch-cases */
@@ -839,7 +871,7 @@ Config.prototype.logUnknown = function logUnknown(json, key) {
 /**
  * Gets the user set host display name. If not provided, it returns the default value.
  *
- * This function is written is this strange way because of the use of caching variables.
+ * This function is written in this strange way because of the use of caching variables.
  * I wanted to cache the DisplayHost, but if I attached the variable to the config object,
  * it sends the extra variable to New Relic, which is not desired.
  *

package/lib/errors/error-collector.js

@@ -35,7 +35,7 @@ class ErrorCollector {
     this.seenObjectsByTransaction = Object.create(null)
     this.seenStringsByTransaction = Object.create(null)
 
-    this.traceAggregator.on('starting error_data data send.', this._onSendErrorTrace.bind(this))
+    this.traceAggregator.on('starting_data_send-error_data', this._onSendErrorTrace.bind(this))
 
     this.errorGroupCallback = null
   }

package/lib/harvester.js

@@ -37,7 +37,7 @@ module.exports = class Harvester {
       this.aggregators.map((aggregator) => {
         return new Promise((resolve) => {
           if (aggregator.enabled) {
-            aggregator.once(`finished ${aggregator.method} data send.`, function finish() {
+            aggregator.once(`finished_data_send-${aggregator.method}`, function finish() {
               resolve()
             })
 

package/lib/metrics/names.js

@@ -204,11 +204,12 @@ const HAPI = {
 
 const UTILIZATION = {
   AWS_ERROR: SUPPORTABILITY.UTILIZATION + '/aws/error',
-  PCF_ERROR: SUPPORTABILITY.UTILIZATION + '/pcf/error',
   AZURE_ERROR: SUPPORTABILITY.UTILIZATION + '/azure/error',
-  GCP_ERROR: SUPPORTABILITY.UTILIZATION + '/gcp/error',
+  BOOT_ID_ERROR: SUPPORTABILITY.UTILIZATION + '/boot_id/error',
   DOCKER_ERROR: SUPPORTABILITY.UTILIZATION + '/docker/error',
-  BOOT_ID_ERROR: SUPPORTABILITY.UTILIZATION + '/boot_id/error'
+  ECS_CONTAINER_ERROR: SUPPORTABILITY.UTILIZATION + '/ecs/container_id/error',
+  GCP_ERROR: SUPPORTABILITY.UTILIZATION + '/gcp/error',
+  PCF_ERROR: SUPPORTABILITY.UTILIZATION + '/pcf/error'
 }
 
 const CUSTOM_EVENTS = {

package/lib/metrics/normalizer.js

@@ -32,6 +32,13 @@ function plain(normalized, path) {
   return path
 }
 
+/**
+ * @event MetricNormalizer#appliedRule
+ * @param {object} rule The rule that matched and was applied.
+ * @param {string} normalized The newly updated metric name.
+ * @param {string} last The metric name that matched the rule.
+ */
+
 /**
  * The collector keeps track of rules that should be applied to metric names,
  * and sends these rules to the agent at connection time. These rules can
@@ -206,6 +213,8 @@ MetricNormalizer.prototype.addSimple = function addSimple(pattern, name) {
  *
  * @param {string} path - The URL path to turn into a name.
  * @returns {NormalizationResults} - The results of normalization.
+ *
+ * @fires MetricNormalizer#appliedRule
  */
 MetricNormalizer.prototype.normalize = function normalize(path) {
   let last = path

package/lib/shim/shim.js

@@ -1300,7 +1300,7 @@ function getName(obj) {
  * @returns {boolean} True if the object is an Object, else false.
  */
 function isObject(obj) {
-  return obj instanceof Object
+  return obj != null && (obj instanceof Object || (!obj.constructor && typeof obj === 'object'))
 }
 
 /**

package/lib/shimmer.js

@@ -286,7 +286,11 @@ const shimmer = (module.exports = {
    */
   registerThirdPartyInstrumentation(agent) {
     for (const [moduleName, instrInfo] of Object.entries(INSTRUMENTATIONS)) {
-      if (instrInfo.module) {
+      if (agent.config.instrumentation?.[moduleName]?.enabled === false) {
+        logger.warn(
+          `Instrumentation for ${moduleName} has been disabled via 'config.instrumentation.${moduleName}.enabled. Not instrumenting package`
+        )
+      } else if (instrInfo.module) {
         // Because external instrumentations can change independent of
         // the agent core, we don't want breakages in them to entirely
         // disable the agent.

package/lib/spans/streaming-span-event-aggregator.js

@@ -13,8 +13,23 @@ const SEND_WARNING =
   'send() is not currently supported on streaming span event aggregator. ' +
   'This warning will not appear again this agent run.'
 
+/**
+ * Indicates that span streaming has begun.
+ *
+ * @event StreamingSpanEventAggregator#started
+ */
+
+/**
+ * Indicates that span streaming has finished.
+ *
+ * @event StreamingSpanEventAggregator#stopped
+ */
+
 // TODO: this doesn't "aggregate". Perhaps we need a different terminology
 // for the base-class and then this implementation can avoid the misleading language.
+/**
+ * Handles streaming of spans to the New Relic data collector.
+ */
 class StreamingSpanEventAggregator extends Aggregator {
   constructor(opts, agent) {
     const { metrics, collector, harvester } = agent
@@ -31,6 +46,11 @@ class StreamingSpanEventAggregator extends Aggregator {
     this.isStream = true
   }
 
+  /**
+   * Start streaming spans to the collector.
+   *
+   * @fires StreamingSpanEventAggregator#started
+   */
   start() {
     if (this.started) {
       return
@@ -43,6 +63,11 @@ class StreamingSpanEventAggregator extends Aggregator {
     this.emit('started')
   }
 
+  /**
+   * Cease streaming of spans to the collector.
+   *
+   * @fires StreamingSpanEventAggregator#stopped
+   */
   stop() {
     if (!this.started) {
       return
@@ -60,7 +85,7 @@ class StreamingSpanEventAggregator extends Aggregator {
       logger.warnOnce('SEND_WARNING', SEND_WARNING)
     }
 
-    this.emit(`finished ${this.method} data send.`)
+    this.emit(`finished_data_send-${this.method}`)
   }
 
   /**
@@ -76,7 +101,7 @@ class StreamingSpanEventAggregator extends Aggregator {
    * Attempts to add the given segment to the collection.
    *
    * @param {TraceSegment}  segment         - The segment to add.
-   * @param {string}        [parentId=null] - The GUID of the parent span.
+   * @param {string}        [parentId] - The GUID of the parent span.
    * @param isRoot
    * @returns {boolean} True if the segment was added, or false if it was discarded.
    */

package/lib/transaction/index.js

@@ -70,6 +70,8 @@ const MULTIPLE_INSERT_MESSAGE =
  * transaction.
  *
  * @param {object} agent The agent.
+ *
+ * @fires Agent#transactionStarted
  */
 function Transaction(agent) {
   if (!agent) {
@@ -200,6 +202,8 @@ Transaction.prototype.isActive = function isActive() {
  * instances of this transaction annotated onto the call stack.
  *
  * @returns {(Transaction|undefined)} this transaction, or undefined
+ *
+ * @fires Agent#transactionFinished
  */
 Transaction.prototype.end = function end() {
   if (!this.timer.isActive()) {

package/lib/transaction/transaction-event-aggregator.js

@@ -48,7 +48,7 @@ class TransactionEventAggregator extends EventAggregator {
     }
 
     // TODO: log?
-    this.emit(`starting ${this.method} data send.`)
+    this.emit(`starting_data_send-${this.method}`)
 
     logger.debug('Splitting transaction events into multiple payloads')
 
@@ -60,7 +60,7 @@ class TransactionEventAggregator extends EventAggregator {
 
     this._sendMultiple(eventPayloadPairs, () => {
       // TODO: Log?
-      this.emit(`finished ${this.method} data send.`)
+      this.emit(`finished_data_send-${this.method}`)
     })
   }
 

package/lib/util/stream-sink.js

@@ -8,6 +8,19 @@
 const EventEmitter = require('events').EventEmitter
 const util = require('util')
 
+/**
+ * Triggered when the stream has been terminated.
+ *
+ * @event StreamSink#close
+ */
+
+/**
+ * Triggered when the stream cannot be written to.
+ *
+ * @event StreamSink#error
+ * @param {Error} error The error that has triggered the event.
+ */
+
 /**
  * Pipe a readable stream into this sink that fulfills the Writable Stream
  * contract and the callback will be fired when the stream has been completely
@@ -30,6 +43,14 @@ function StreamSink(callback) {
 }
 util.inherits(StreamSink, EventEmitter)
 
+/**
+ * Write data to the stream.
+ *
+ * @param {string} string The data to write.
+ * @returns {boolean}
+ *
+ * @fires StreamSink#error
+ */
 StreamSink.prototype.write = function write(string) {
   if (!this.writable) {
     this.emit('error', new Error('Sink no longer writable!'))
@@ -49,6 +70,11 @@ StreamSink.prototype.end = function end() {
   this.callback(null, this.sink)
 }
 
+/**
+ * Stop the stream and mark it unusable.
+ *
+ * @fires StreamSink#close
+ */
 StreamSink.prototype.destroy = function destroy() {
   this.emit('close')
   this.writable = false

package/lib/utilization/docker-info.js

@@ -6,7 +6,6 @@
 'use strict'
 
 const fs = require('node:fs')
-const http = require('node:http')
 const log = require('../logger').child({ component: 'docker-info' })
 const common = require('./common')
 const NAMES = require('../metrics/names')
@@ -17,12 +16,17 @@ const CGROUPS_V1_PATH = '/proc/self/cgroup'
 const CGROUPS_V2_PATH = '/proc/self/mountinfo'
 const BOOT_ID_PROC_FILE = '/proc/sys/kernel/random/boot_id'
 
-module.exports.getVendorInfo = fetchDockerVendorInfo
-module.exports.clearVendorCache = function clearDockerVendorCache() {
+module.exports = {
+  clearVendorCache: clearDockerVendorCache,
+  getBootId,
+  getVendorInfo: fetchDockerVendorInfo
+}
+
+function clearDockerVendorCache() {
   vendorInfo = null
 }
 
-module.exports.getBootId = function getBootId(agent, callback, logger = log) {
+function getBootId(agent, callback, logger = log) {
   if (!/linux/i.test(os.platform())) {
     logger.debug('Platform is not a flavor of linux, omitting boot info')
     return setImmediate(callback, null, null)
@@ -37,76 +41,8 @@ module.exports.getBootId = function getBootId(agent, callback, logger = log) {
     }
 
     logger.debug('Container boot id is not available in cgroups info')
-
-    if (hasAwsContainerApi() === false) {
-      // We don't seem to have a recognized location for getting the container
-      // identifier.
-      logger.debug('Container is not in a recognized ECS container, omitting boot info')
-      recordBootIdError(agent)
-      return callback(null, null)
-    }
-
-    getEcsContainerId({ agent, callback, logger })
-  })
-}
-
-/**
- * Queries the AWS ECS metadata API to get the boot id.
- *
- * @param {object} params Function parameters.
- * @param {object} params.agent Newrelic agent instance.
- * @param {Function} params.callback Typical error first callback. The second
- * parameter is the boot id as a string.
- * @param {object} [params.logger] Internal logger instance.
- */
-function getEcsContainerId({ agent, callback, logger }) {
-  const ecsApiUrl =
-    process.env.ECS_CONTAINER_METADATA_URI_V4 || process.env.ECS_CONTAINER_METADATA_URI
-  const req = http.request(ecsApiUrl, (res) => {
-    let body = Buffer.alloc(0)
-    res.on('data', (chunk) => {
-      body = Buffer.concat([body, chunk])
-    })
-    res.on('end', () => {
-      try {
-        const json = body.toString('utf8')
-        const data = JSON.parse(json)
-        if (data.DockerId == null) {
-          logger.debug('Failed to find DockerId in response, omitting boot info')
-          recordBootIdError(agent)
-          return callback(null, null)
-        }
-        callback(null, data.DockerId)
-      } catch (error) {
-        logger.debug('Failed to process ECS API response, omitting boot info: ' + error.message)
-        recordBootIdError(agent)
-        callback(null, null)
-      }
-    })
-  })
-
-  req.on('error', () => {
-    logger.debug('Failed to query ECS endpoint, omitting boot info')
-    recordBootIdError(agent)
     callback(null, null)
   })
-
-  req.end()
-}
-
-/**
- * Inspects the running environment to determine if the AWS ECS metadata API
- * is available.
- *
- * @see https://docs.aws.amazon.com/AmazonECS/latest/developerguide/ec2-metadata.html
- *
- * @returns {boolean}
- */
-function hasAwsContainerApi() {
-  if (process.env.ECS_CONTAINER_METADATA_URI_V4 != null) {
-    return true
-  }
-  return process.env.ECS_CONTAINER_METADATA_URI != null
 }
 
 /**

package/lib/utilization/ecs-info.js

@@ -0,0 +1,112 @@
+/*
+ * Copyright 2024 New Relic Corporation. All rights reserved.
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+'use strict'
+
+const http = require('node:http')
+const NAMES = require('../metrics/names')
+
+module.exports = function fetchEcsInfo(
+  agent,
+  callback,
+  {
+    logger = require('../logger').child({ component: 'ecs-info' }),
+    getEcsContainerId = _getEcsContainerId,
+    hasAwsContainerApi = _hasAwsContainerApi,
+    recordIdError = _recordIdError
+  } = {}
+) {
+  // Per spec, we do not have a `detect_ecs` key. Since ECS is a service of AWS,
+  // we rely on the `detect_aws` setting.
+  if (!agent.config.utilization || !agent.config.utilization.detect_aws) {
+    return setImmediate(callback, null)
+  }
+
+  if (hasAwsContainerApi() === false) {
+    logger.debug('ECS API not available, omitting ECS container id info')
+    recordIdError(agent)
+    return callback(null, null)
+  }
+
+  getEcsContainerId({
+    agent,
+    logger,
+    recordIdError,
+    callback: (error, dockerId) => {
+      if (error) {
+        return callback(error, null)
+      }
+      if (dockerId === null) {
+        // Some error happened where we could not find the id. Skipping.
+        return callback(null, null)
+      }
+      return callback(null, { ecsDockerId: dockerId })
+    }
+  })
+}
+
+/**
+ * Queries the AWS ECS metadata API to get the boot id.
+ *
+ * @param {object} params Function parameters.
+ * @param {object} params.agent Newrelic agent instance.
+ * @param {Function} params.callback Typical error first callback. The second
+ * parameter is the boot id as a string.
+ * @param {object} params.logger Internal logger instance.
+ * @param {function} params.recordIdError Function to record error metric.
+ */
+function _getEcsContainerId({ agent, callback, logger, recordIdError }) {
+  const ecsApiUrl =
+    process.env.ECS_CONTAINER_METADATA_URI_V4 || process.env.ECS_CONTAINER_METADATA_URI
+  const req = http.request(ecsApiUrl, (res) => {
+    let body = Buffer.alloc(0)
+    res.on('data', (chunk) => {
+      body = Buffer.concat([body, chunk])
+    })
+    res.on('end', () => {
+      try {
+        const json = body.toString('utf8')
+        const data = JSON.parse(json)
+        if (data.DockerId == null) {
+          logger.debug('Failed to find DockerId in response, omitting boot info')
+          recordIdError(agent)
+          return callback(null, null)
+        }
+        callback(null, data.DockerId)
+      } catch (error) {
+        logger.debug('Failed to process ECS API response, omitting boot info: ' + error.message)
+        recordIdError(agent)
+        callback(null, null)
+      }
+    })
+  })
+
+  req.on('error', () => {
+    logger.debug('Failed to query ECS endpoint, omitting boot info')
+    recordIdError(agent)
+    callback(null, null)
+  })
+
+  req.end()
+}
+
+/**
+ * Inspects the running environment to determine if the AWS ECS metadata API
+ * is available.
+ *
+ * @see https://docs.aws.amazon.com/AmazonECS/latest/developerguide/ec2-metadata.html
+ *
+ * @returns {boolean}
+ */
+function _hasAwsContainerApi() {
+  if (process.env.ECS_CONTAINER_METADATA_URI_V4 != null) {
+    return true
+  }
+  return process.env.ECS_CONTAINER_METADATA_URI != null
+}
+
+function _recordIdError(agent) {
+  agent.metrics.getOrCreateMetric(NAMES.UTILIZATION.ECS_CONTAINER_ERROR).incrementCallCount()
+}

package/lib/utilization/index.js

@@ -9,11 +9,12 @@ const logger = require('../logger').child({ component: 'utilization' })
 
 const VENDOR_METHODS = {
   aws: require('./aws-info'),
-  pcf: require('./pcf-info'),
   azure: require('./azure-info'),
-  gcp: require('./gcp-info'),
   docker: require('./docker-info').getVendorInfo,
-  kubernetes: require('./kubernetes-info')
+  ecs: require('./ecs-info'),
+  gcp: require('./gcp-info'),
+  kubernetes: require('./kubernetes-info'),
+  pcf: require('./pcf-info')
 }
 const VENDOR_NAMES = Object.keys(VENDOR_METHODS)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newrelic",
-  "version": "12.3.1",
+  "version": "12.5.0",
   "author": "New Relic Node.js agent team <nodejs@newrelic.com>",
   "license": "Apache-2.0",
   "contributors": [
@@ -219,6 +219,7 @@
     "@aws-sdk/client-s3": "^3.556.0",
     "@aws-sdk/s3-request-presigner": "^3.556.0",
     "@koa/router": "^12.0.1",
+    "@matteo.collina/tspl": "^0.1.1",
     "@newrelic/eslint-config": "^0.3.0",
     "@newrelic/newrelic-oss-cli": "^0.1.2",
     "@newrelic/test-utilities": "^9.1.0",
@@ -254,7 +255,6 @@
     "nock": "11.8.0",
     "proxy": "^2.1.1",
     "proxyquire": "^1.8.0",
-    "rfdc": "^1.3.1",
     "rimraf": "^2.6.3",
     "self-cert": "^2.0.0",
     "should": "*",