newrelic 9.0.0 → 9.0.1

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

@@ -5,7 +5,6 @@
 
 'use strict'
 
-const async = require('async')
 const logger = require('../logger').child({ component: 'transaction-event-aggregator' })
 const EventAggregator = require('../aggregators/event-aggregator')
 
@@ -100,20 +99,19 @@ class TransactionEventAggregator extends EventAggregator {
     return rawEvent.value
   }
 
-  _sendMultiple(eventPayloadPairs, sendCallback) {
+  async _sendMultiple(eventPayloadPairs, sendCallback) {
     const self = this
 
     // Send payloads one at a time
-    async.eachOfSeries(
-      eventPayloadPairs,
-      (payloadPair, index, asyncCallback) => {
-        logger.debug(
-          'Sending payload %d of %d to %s',
-          index + 1,
-          eventPayloadPairs.length,
-          self.method
-        )
-
+    const promises = eventPayloadPairs.map((payloadPair, index) => {
+      logger.debug(
+        'Sending payload %d of %d to %s',
+        index + 1,
+        eventPayloadPairs.length,
+        self.method
+      )
+
+      return new Promise((resolve) => {
         self._sendSplitPayload(payloadPair.rawData, payloadPair.payload, (error) => {
           if (error) {
             logger.warn(error, 'An error occurred sending payload')
@@ -126,16 +124,16 @@ class TransactionEventAggregator extends EventAggregator {
             self.method
           )
 
-          // don't pass on error, allow next payload to attempt to send
-          asyncCallback()
+          // swallow error, allow all payloads to attempt to send
+          resolve()
         })
-      },
-      function afterAllPayloadsSent() {
-        logger.debug('Finished sending %d payloads to %s', eventPayloadPairs.length, self.method)
+      })
+    })
 
-        sendCallback()
-      }
-    )
+    await Promise.all(promises)
+    logger.debug('Finished sending %d payloads to %s', eventPayloadPairs.length, self.method)
+
+    sendCallback()
   }
 
   _sendSplitPayload(rawData, payload, callback) {

package/lib/util/application-logging.js

@@ -82,8 +82,10 @@ utils.isLogForwardingEnabled = function isLogForwardingEnabled(config, agent) {
  * @param {object} metrics metrics module
  */
 utils.incrementLoggingLinesMetrics = function incrementLoggingLinesMetrics(level, metrics) {
+  const levelMetric = getLogLevel(level)
+
   metrics.getOrCreateMetric(LOGGING.LINES).incrementCallCount()
-  metrics.getOrCreateMetric(LOGGING.LEVELS[level.toUpperCase()]).incrementCallCount()
+  metrics.getOrCreateMetric(levelMetric).incrementCallCount()
 }
 
 /**
@@ -95,3 +97,14 @@ utils.incrementLoggingLinesMetrics = function incrementLoggingLinesMetrics(level
 utils.createModuleUsageMetric = function createModuleUsageMetric(lib, metrics) {
   metrics.getOrCreateMetric(LOGGING.LIBS[lib.toUpperCase()]).incrementCallCount()
 }
+
+function getLogLevel(level) {
+  if (!level) {
+    return LOGGING.LEVELS.UNKNOWN
+  }
+  const logLevel = LOGGING.LEVELS[level.toUpperCase()]
+  if (!logLevel) {
+    return LOGGING.LEVELS.UNKNOWN
+  }
+  return logLevel
+}

package/lib/util/unwrapped-core.js

@@ -7,5 +7,7 @@
 
 const copy = require('./copy')
 const fs = require('fs')
+const fsPromises = require('fs/promises')
 
 exports.fs = copy.shallow(fs)
+exports.fsPromises = copy.shallow(fsPromises)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newrelic",
-  "version": "9.0.0",
+  "version": "9.0.1",
   "author": "New Relic Node.js agent team <nodejs@newrelic.com>",
   "license": "Apache-2.0",
   "contributors": [
@@ -169,7 +169,6 @@
     "@newrelic/koa": "^7.0.0",
     "@newrelic/superagent": "^6.0.0",
     "@tyriar/fibonacci-heap": "^2.0.7",
-    "async": "^3.2.3",
     "concat-stream": "^2.0.0",
     "https-proxy-agent": "^5.0.0",
     "json-stringify-safe": "^5.0.0",
@@ -188,9 +187,7 @@
     "@octokit/rest": "^18.0.15",
     "@slack/bolt": "^3.7.0",
     "ajv": "^6.12.6",
-    "architect": "*",
-    "benchmark": "^2.1.4",
-    "bluebird": "^3.4.7",
+    "async": "^3.2.4",
     "chai": "^4.1.2",
     "commander": "^7.0.0",
     "eslint": "^7.32.0",
@@ -201,22 +198,17 @@
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^3.4.0",
     "express": "*",
-    "generic-pool": "^3.6.1",
     "glob": "^7.1.2",
     "got": "^11.8.5",
     "husky": "^6.0.0",
     "jsdoc": "^3.6.3",
     "lint-staged": "^11.0.0",
     "memcached": ">=0.2.8",
-    "minami": "^1.1.1",
-    "mongodb": "^3.3.3",
-    "mysql": "*",
     "nock": "11.8.0",
     "prettier": "^2.3.2",
     "proxyquire": "^1.8.0",
     "q": "*",
-    "redis": "^1.0.0",
-    "request": "^2.88.0",
+    "request": "^2.88.2",
     "rimraf": "^2.6.3",
     "should": "*",
     "sinon": "^4.5.0",