net-snmp 3.8.3 → 3.9.0

package/README.md

@@ -517,6 +517,14 @@ supplemental information. An authentication error, for example -- code
 `ResponseInvalidCode.EAuthFailure` -- will contain a map in `info`
 with the attempted authentication data which failed to authenticate.
 
+## snmp.ProcessingError
+
+If a receiver or an agent receives a packet it is unable to decode,
+then it will produce a `ProcessingError` containing:
+* `rinfo` information on the origin of the packet,
+* a `buffer` containing the packet contents, and
+* an `error` containing the original error encountered during processing.
+
 # Using This Module: Command & Notification Generator
 
 This library provides a `Session` class to provide support for building
@@ -3227,6 +3235,14 @@ Example programs are included under the module's `example` directory.
 
  * Fix incorrect user level assignment
 
+## Version 3.8.4 - 29/09/2022
+
+ * Fix IpAddress byte array set request handling
+
+## Version 3.9.0 - 11/12/2022
+
+ * Add ProcessingError to handle agent/receiver decode failures
+
 # License
 
 Copyright (c) 2020 Mark Abrahams <mark@abrahams.co.nz>

package/example/snmp-receiver.js

@@ -15,7 +15,8 @@ var cb = function(error, trap) {
     var now = new Date().toLocaleString();
     var trapType;
     if (error) {
-        console.log(now + ": " + error.message);
+        console.log (now + ": " + error.message);
+        console.error (error);
     } else {
         trapType = snmp.PduType[trap.pdu.type] || "Unknown";
         if ( verbose ) {

package/index.js

@@ -315,6 +315,16 @@ function RequestTimedOutError (message) {
 }
 util.inherits (RequestTimedOutError, Error);
 
+function ProcessingError (message, error, rinfo, buffer) {
+	this.name = "ProcessingError";
+	this.message = message;
+	this.error = error;
+	this.rinfo = rinfo;
+	this.buffer = buffer;
+	Error.captureStackTrace(this, ProcessingError);
+}
+util.inherits (ProcessingError, Error);
+
 /*****************************************************************************
  ** OID and varbind helper functions
  **/
@@ -3071,8 +3081,15 @@ Receiver.prototype.getAuthorizer = function () {
 };
 
 Receiver.prototype.onMsg = function (buffer, rinfo) {
-	var message = Listener.processIncoming (buffer, this.authorizer, this.callback);
-	var reportMessage;
+
+	let message;
+
+	try {
+		message = Listener.processIncoming (buffer, this.authorizer, this.callback);
+	} catch (error) {
+		this.callback (new ProcessingError ("Failure to process incoming message", error, rinfo, buffer));
+		return;
+	}
 
 	if ( ! message ) {
 		return;
@@ -3090,7 +3107,7 @@ Receiver.prototype.onMsg = function (buffer, rinfo) {
 			this.callback (new RequestInvalidError ("Only discovery GetRequests are supported and this message does not have the reportable flag set"));
 			return;
 		}
-		reportMessage = message.createReportResponseMessage (this.engine, this.context);
+		let reportMessage = message.createReportResponseMessage (this.engine, this.context);
 		this.listener.send (reportMessage, rinfo);
 		return;
 	}
@@ -4536,8 +4553,15 @@ Agent.prototype.tableRowStatusHandlerInternal = function (createRequest) {
 };
 
 Agent.prototype.onMsg = function (buffer, rinfo) {
-	var message = Listener.processIncoming (buffer, this.authorizer, this.callback);
-	var reportMessage;
+
+	let message;
+
+	try {
+		message = Listener.processIncoming (buffer, this.authorizer, this.callback);
+	} catch (error) {
+		this.callback (new ProcessingError ("Failure to process incoming message", error, rinfo, buffer));
+		return;
+	}
 
 	if ( ! message ) {
 		return;
@@ -4546,7 +4570,7 @@ Agent.prototype.onMsg = function (buffer, rinfo) {
 	// SNMPv3 discovery
 	if ( message.version == Version3 && message.pdu.type == PduType.GetRequest &&
 			! message.hasAuthoritativeEngineID() && message.isReportable () ) {
-		reportMessage = message.createReportResponseMessage (this.engine, this.context);
+		let reportMessage = message.createReportResponseMessage (this.engine, this.context);
 		this.listener.send (reportMessage, rinfo);
 		return;
 	}
@@ -4603,7 +4627,8 @@ Agent.prototype.castSetValue = function ( type, value ) {
 
 		case ObjectType.IpAddress:
 			// A 32-bit internet address represented as OCTET STRING of length 4
-			if ( typeof value != "string" || value.length != 4 ) {
+			var bytes = value.split(".");
+			if ( typeof value != "string" || bytes.length != 4 ) {
 				throw new Error("Invalid IpAddress", value);
 			}
 			return value;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "net-snmp",
-  "version": "3.8.3",
+  "version": "3.9.0",
   "description": "JavaScript implementation of the Simple Network Management Protocol (SNMP)",
   "main": "index.js",
   "directories": {