@newrelic/security-agent 0.3.0 → 0.4.0

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+### v0.4.0 (2023-10-27)
+#### Features
+* Added event stats for RASP, IAST and exit events in healthcheck. 
+#### Miscellaneous chores
+* Deps: Updated uuid to v9.0.1 and @aws-sdk/client-lambda to v3.405.0
+* Upgraded axios to 1.6.0 to resolve CVE-2023-45857 
+#### Continuous integration
+* Removed Node.js 14.x from CI.
+* Added should_skip flag to skip CI based on label.
+
 ### v0.3.0 (2023-09-25)
 #### Features
 * Last leg acknowledgement in IAST scanning.

package/THIRD_PARTY_NOTICES.md

@@ -75,7 +75,7 @@ code, the source code can be found at [https://github.com/newrelic/csec-node-age
 
 ### @aws-sdk/client-lambda
 
-This product includes source derived from [@aws-sdk/client-lambda](https://github.com/aws/aws-sdk-js-v3) ([v3.379.1](https://github.com/aws/aws-sdk-js-v3/tree/v3.379.1)), distributed under the [Apache-2.0 License](https://github.com/aws/aws-sdk-js-v3/blob/v3.379.1/LICENSE):
+This product includes source derived from [@aws-sdk/client-lambda](https://github.com/aws/aws-sdk-js-v3) ([v3.435.0](https://github.com/aws/aws-sdk-js-v3/tree/v3.435.0)), distributed under the [Apache-2.0 License](https://github.com/aws/aws-sdk-js-v3/blob/v3.435.0/LICENSE):
 
 ```
                                 Apache License
@@ -284,28 +284,16 @@ This product includes source derived from [@aws-sdk/client-lambda](https://githu
 
 ### axios
 
-This product includes source derived from [axios](https://github.com/axios/axios) ([v0.21.4](https://github.com/axios/axios/tree/v0.21.4)), distributed under the [MIT License](https://github.com/axios/axios/blob/v0.21.4/LICENSE):
+This product includes source derived from [axios](https://github.com/axios/axios) ([v1.6.0](https://github.com/axios/axios/tree/v1.6.0)), distributed under the [MIT License](https://github.com/axios/axios/blob/v1.6.0/LICENSE):
 
 ```
-Copyright (c) 2014-present Matt Zabriskie
+# Copyright (c) 2014-present Matt Zabriskie & Collaborators
 
-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:
+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 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.
+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.
 
 ```
 
@@ -790,7 +778,7 @@ THE SOFTWARE.
 
 ### uuid
 
-This product includes source derived from [uuid](https://github.com/uuidjs/uuid) ([v9.0.0](https://github.com/uuidjs/uuid/tree/v9.0.0)), distributed under the [MIT License](https://github.com/uuidjs/uuid/blob/v9.0.0/LICENSE.md):
+This product includes source derived from [uuid](https://github.com/uuidjs/uuid) ([v9.0.1](https://github.com/uuidjs/uuid/tree/v9.0.1)), distributed under the [MIT License](https://github.com/uuidjs/uuid/blob/v9.0.1/LICENSE.md):
 
 ```
 The MIT License (MIT)

package/lib/nr-security-agent/lib/core/connections/websocket/response/fuzz-request-handler.js

@@ -180,7 +180,10 @@ function handleFuzzResponse(response, fuzzDetails) {
  */
 function parseAxiosHttpRequestToFuzz(requestObject) {
     let serverName = requestObject.serverName ? requestObject.serverName : LOCALHOST;
-    let host = serverName + COLON + requestObject.serverPort
+    let host = serverName + COLON + requestObject.serverPort;
+    if(requestObject.headers && requestObject.headers['content-length']){
+        delete requestObject.headers['content-length'];
+    }
     return {
         url: requestObject.protocol + COLON_SLASH_SLASH + host + requestObject.url,
         method: requestObject.method,

package/lib/nr-security-agent/lib/core/connections/websocket/websocket.js

@@ -17,7 +17,7 @@ const hc = require('../../health-check');
 const fs = require('fs');
 const { promisify } = require('../../sec-util');
 const ResponseHandler = require('./response');
-const { EMPTY_APPLICATION_UUID, LOG_MESSAGES, CSEC_SEP, NR_CSEC_FUZZ_REQUEST_ID, EXITEVENT,JSON_NAME, RASP } = require('../../sec-agent-constants');
+const { EMPTY_APPLICATION_UUID, LOG_MESSAGES, CSEC_SEP, NR_CSEC_FUZZ_REQUEST_ID, EXITEVENT, JSON_NAME, RASP } = require('../../sec-agent-constants');
 const statusUtils = require('../../statusUtils');
 const commonUtils = require('../../commonUtils');
 const IASTUtil = require('../websocket/response/IASTUtils');
@@ -228,12 +228,14 @@ SecWebSocket.prototype.dispatch = async function dispatch(event) {
         await promisify(this.instance, this.instance.send)(eventStr, { mask: true }, () => {
             logger.debug(LOG_MESSAGES.EVENT_SENT + eventStr);
             if (event.jsonName == JSON_NAME.EVENT) {
-                if (NRAgent.config.security.mode == RASP) {
-                    hc.getInstance().raspEventStats.sent++;
+
+                if (event.httpRequest && event.httpRequest.headers[NR_CSEC_FUZZ_REQUEST_ID]) {
+                    hc.getInstance().iastEventStats.sent++;
                 }
                 else {
-                    hc.getInstance().iastEventStats.sent++;
+                    hc.getInstance().raspEventStats.sent++;
                 }
+
             }
             if (event.jsonName == EXITEVENT) {
                 hc.getInstance().exitEventStats.sent++;
@@ -318,12 +320,13 @@ const handleDispatchFailure = (self, event, eventStr) => {
     // so feeding an old applicationinfo JSON would be incorrect
     if (eventStr && event.jsonName !== APPLICATION_INFO_JSON_NAME) {
         hc.getInstance().registerEventDrop();
-        if (NRAgent.config.security.mode == "RASP") {
-            hc.getInstance().raspEventStats.errorCount++;
+        if (event.httpRequest && event.httpRequest.headers[NR_CSEC_FUZZ_REQUEST_ID]) {
+            hc.getInstance().iastEventStats.errorCount++;
         }
         else {
-            hc.getInstance().iastEventStats.errorCount++;
+            hc.getInstance().raspEventStats.errorCount++;
         }
+
     }
 };
 

package/lib/nr-security-agent/lib/core/sec-util.js

@@ -66,13 +66,13 @@ const sleep = (timeout) => new Promise((resolve) => {
 function generateSecEvent(securityMetadata) {
     const traceObject = securityMetadata.traceObject;
     const request = securityMetadata.request;
-    if (NRAgent.config.security.mode == RASP) {
-        HC.getInstance().raspEventStats.processed++;
-    }
-    else {
+
+    if(request && request.headers[NR_CSEC_FUZZ_REQUEST_ID]){
         HC.getInstance().iastEventStats.processed++;
     }
-
+    else{
+        HC.getInstance().raspEventStats.processed++;
+    }
     try {
         let stakTrace = traceObject.stacktrace;
         let uri = EMPTY_STR;
@@ -102,12 +102,13 @@ function generateSecEvent(securityMetadata) {
     } catch (e) {
         statusUtils.addErrortoBuffer(e);
         logger.error("Error in generating event:", e);
-        if (NRAgent.config.security.mode == RASP) {
-            HC.getInstance().raspEventStats.errorCount++;
-        }
-        else {
+       
+        if(request && request.headers[NR_CSEC_FUZZ_REQUEST_ID]){
             HC.getInstance().iastEventStats.errorCount++;
         }
+        else{
+            HC.getInstance().raspEventStats.errorCount++;
+        }
     }
 
 }

package/lib/nr-security-agent/lib/core/websocket-client.js

@@ -23,7 +23,7 @@ const {
     }
 } = require('./connections/websocket');
 const { HC_INTERVAL_MS } = require('../../resources/config');
-const { LOG_DIR } = require('./sec-agent-constants');
+const { LOG_DIR, EXITEVENT, JSON_NAME } = require('./sec-agent-constants');
 const statusUtils = require('./statusUtils');
 const njsAgentConstants = require('./sec-agent-constants');
 
@@ -100,6 +100,7 @@ const sendHC = async function () {
         json = JSON.parse(json);
         json.policyVersion = applicationInfo.policyVersion;
         json.entityGuid = applicationInfo.entityGuid;
+
         try {
             json.stats = await commonUtils.getHCStats();
         } catch (error) {
@@ -114,6 +115,12 @@ const sendHC = async function () {
         json.serviceStatus.agentActiveStat = commonUtils.isAgentActiveState();
         json.serviceStatus.iastRestClient = commonUtils.iastRestClientStatus();
 
+        json.exitEventSentCount = json.exitEventStats.sent;
+        json.eventRejectionCount = json.raspEventStats.rejected + json.iastEventStats.rejected;
+        json.eventProcessingErrorCount = json.raspEventStats.errorCount + json.iastEventStats.errorCount;
+        json.eventSendRejectionCount = json.eventRejectionCount;
+        json.eventSendErrorCount = json.eventProcessingErrorCount;
+
         hc.getInstance().resetDropCount();
         hc.getInstance().resetProcessedCount();
         hc.getInstance().resetEventSentCount();
@@ -134,13 +141,13 @@ const sendHC = async function () {
  */
 function getDispatcherAndSendEvent(eventJson) {
     const eventType = eventJson.jsonName;
-    if (eventType === 'Event') {
+    if (eventType === JSON_NAME.EVENT || eventType === EXITEVENT) {
         hc.getInstance().registerEventProcessed();
     }
 
     dispatcher(eventJson);
 
-    if (eventType === 'Event') {
+    if (eventType === JSON_NAME.EVENT || eventType === EXITEVENT) {
         hc.getInstance().registerEventSent();
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newrelic/security-agent",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "New Relic Security Agent for Node.js",
   "main": "index.js",
   "jsonVersion": "1.1.0",
@@ -35,8 +35,8 @@
     "url": "https://github.com/newrelic/csec-node-agent.git"
   },
   "dependencies": {
-    "@aws-sdk/client-lambda": "^3.363.0",
-    "axios": "0.21.4",
+    "@aws-sdk/client-lambda": "^3.405.0",
+    "axios": "1.6.0",
     "check-disk-space": "3.3.1",
     "content-type": "^1.0.5",
     "fast-safe-stringify": "^2.1.1",
@@ -55,7 +55,7 @@
     "sync-request": "^6.1.0",
     "unescape": "^1.0.1",
     "unescape-js": "^1.1.4",
-    "uuid": "^9.0.0",
+    "uuid": "^9.0.1",
     "ws": "^7.5.9"
   },
   "devDependencies": {