newrelic 14.2.0 → 14.2.1

package/NEWS.md

@@ -1,3 +1,26 @@
+### v14.2.1 (2026-07-09)
+
+#### Bug fixes
+
+* Updated `winston` instrumenting to assign `handleExceptions` to the first NrTransport to avoid registering duplicate `uncaughtException` handlers ([#4108](https://github.com/newrelic/node-newrelic/pull/4108)) ([d60a48d](https://github.com/newrelic/node-newrelic/commit/d60a48d3b9399d3a084c64cf53c3a62c7a4cab0b))
+  * This would create a linear memory leak because the `NrTransport` objects for loggers no longer in use, would not be freed because of the global `process.on('uncaughtException')` handler
+
+#### Documentation
+
+* Updated compatibility report ([#4099](https://github.com/newrelic/node-newrelic/pull/4099)) ([058fde6](https://github.com/newrelic/node-newrelic/commit/058fde6d64e3606b82875380ba51f9bc44a4b994))
+
+#### Continuous integration
+
+* Pinned CI npm to 11 so native versioned suites build on Node 22 ([#4114](https://github.com/newrelic/node-newrelic/pull/4114)) ([b844e94](https://github.com/newrelic/node-newrelic/commit/b844e94317e573410d96f8971308b2d1535570ca))
+* Removed the 8 minute versioned shard timeout ([#4111](https://github.com/newrelic/node-newrelic/pull/4111)) ([0ef6374](https://github.com/newrelic/node-newrelic/commit/0ef63749b2739d7c7446a6b7c6d3ee80d2d360e5))
+* Attempted to fix memcached CI issue again ([#4098](https://github.com/newrelic/node-newrelic/pull/4098)) ([f67fef5](https://github.com/newrelic/node-newrelic/commit/f67fef5febe71ffab853164aba06c76cd2c6c26d))
+* Shard versioned tests across CI runners ([#4101](https://github.com/newrelic/node-newrelic/pull/4101)) ([1471734](https://github.com/newrelic/node-newrelic/commit/14717345d87ec46a4eba18c513e10e3caed40f7c))
+
+#### Tests
+
+* Added `samples` to prisma and next to cut down on versions under test on packages we recommend using hybrid ageninstrumentation ([#4112](https://github.com/newrelic/node-newrelic/pull/4112)) ([7d8bd7c](https://github.com/newrelic/node-newrelic/commit/7d8bd7cd2e3076d17ad25b75628956d1158fc23a))
+* Resolved `grpc` `proto-loader` from the running suite's `node_modules` ([#4110](https://github.com/newrelic/node-newrelic/pull/4110)) ([5b5a9f8](https://github.com/newrelic/node-newrelic/commit/5b5a9f8d8e0f17557b9f202b0098e2ec84c9e028))
+
 ### v14.2.0 (2026-07-07)
 
 #### Features

package/lib/subscribers/winston/configure.js

@@ -12,11 +12,18 @@ const NrTransport = require('./nr-winston-transport.js')
  * during initial construction and on any subsequent reconfiguration.
  *
  * Handles `NrTransport` injection (after `configure` runs, via `end`).
+ *
+ * @property {boolean} handlingExceptions Indicates if this instrumentation has
+ * already registered a global exception handler or not. We only need a single
+ * exception handler per process.
  */
 module.exports = class WinstonConfigure extends ApplicationLogsSubscriber {
   constructor({ agent, logger }) {
     super({ agent, logger, channelName: 'nr_configure', packageName: 'winston' })
     this.events = ['end']
+    // we only want to assign `handleExceptions` to the first NrTransport
+    // customers could be constructing multiple logger instances
+    this.handlingExceptions = false
   }
 
   handler(data, ctx) {
@@ -39,11 +46,12 @@ module.exports = class WinstonConfigure extends ApplicationLogsSubscriber {
     const hasNrTransport = logger.transports?.some((t) => t.name === 'newrelic')
     if (!hasNrTransport) {
       const nrTransport = new NrTransport({ agent: this.agent })
-      // Only handle exceptions if the logger has user-configured transports.
-      // This prevents the default logger (created during require('winston')
-      // module init with no transports) from double-handling exceptions.
-      if (logger.transports.length === 0) {
-        nrTransport.handleExceptions = false
+      // assigning `transport.handleExceptions` adds a global `process.on('uncaughtException')`
+      // handler. Doing it for more than one NrTransport instance will cause duplicate log lines
+      if (this.handlingExceptions === false && logger.transports.length > 0) {
+        // See: https://github.com/winstonjs/winston#handling-uncaught-exceptions-with-winston
+        nrTransport.handleExceptions = true
+        this.handlingExceptions = true
       }
       logger.add(nrTransport)
     }

package/lib/subscribers/winston/nr-winston-transport.js

@@ -14,12 +14,15 @@ const { truncate } = require('../../util/application-logging')
  *
  * Note*: This copies the log line so no other transports will get the
  * mutated data.
+ *
+ * @property {string} name name of the transport
+ * @property {Agent} agent the instance of agent
+ * @property {object} config the agent configuration
+ * @property {boolean} handleExceptions set in `lib/subscribers/winston/configure.js` for the first NrTransport instance
+ * to ensure that only one is handling uncaught exceptions in winston transports
  */
 class NrTransport extends TransportStream {
   constructor(opts = {}) {
-    // set this option to have winston handle uncaught exceptions
-    // See: https://github.com/winstonjs/winston#handling-uncaught-exceptions-with-winston
-    opts.handleExceptions = true
     super(opts)
     this.name = 'newrelic'
     this.agent = opts.agent

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newrelic",
-  "version": "14.2.0",
+  "version": "14.2.1",
   "author": "New Relic Node.js agent team <nodejs@newrelic.com>",
   "license": "Apache-2.0",
   "contributors": [
@@ -170,7 +170,7 @@
     "lint:fix": "eslint --fix .",
     "public-docs": "jsdoc -c ./jsdoc-conf.jsonc",
     "publish-docs": "./bin/publish-docs.sh",
-    "services": "DOCKER_PLATFORM=linux/$(uname -m) docker compose up -d --wait",
+    "services": "./bin/start-services.sh",
     "services:start": "npm run services",
     "services:stop": "docker compose down",
     "smoke": "time borp --timeout 180000 --reporter ./test/lib/test-reporter.mjs 'test/smoke/**/*.test.js'",