net-snmp 3.15.1 → 3.15.3

package/.vscode/launch.json

@@ -15,6 +15,7 @@
             "args": [
                 "-v", "2c",
                 "-c", "public",
+                "-s", "5678",
                 "127.0.0.1",
                 "1.3.6.1.2.1.31.1.1.1.6.1"
             ]

package/README.md

@@ -3420,6 +3420,14 @@ Example programs are included under the module's `example` directory.
 
  * Fix bundler failure due to unnecessary string length assignment
 
+## Version 3.15.2 - 03/12/2024
+
+ * Change return of non-existing OIDs under table from NoSuchObject to NoSuchInstance
+
+## Version 3.15.3 - 04/12/2024
+
+ * Fix agent SNMP SetRequest for OIDs
+
 # License
 
 Copyright (c) 2020 Mark Abrahams <mark@abrahams.co.nz>

package/example/option-parser.js

@@ -9,6 +9,7 @@ var user;
 var session;
 var nonRepeaters;
 var maxRepetitions;
+var sourcePort;
 var oids;
 var varbinds;
 var columns;
@@ -44,6 +45,7 @@ if ( snmpOptions.version == snmp.Version3 ) {
 
 nonRepeaters = options.o;
 maxRepetitions = options.r;
+sourcePort = options.s;
 
 if (options._.length < 2) {
 	console.log ("usage: " + process.argv[1] + " [<options>] <target> <oid>");
@@ -77,6 +79,8 @@ if ( command.includes('snmp-trap') || command.includes('snmp-inform') || command
     snmpOptions.port = options.p;
 }
 
+snmpOptions.sourcePort = sourcePort;
+
 if ( snmpOptions.version == snmp.Version3 ) {
 	session = snmp.createV3Session (target, user, snmpOptions);
 } else {

package/example/snmp-get.js

@@ -18,4 +18,9 @@ session.get (oids, function (error, varbinds) {
 				console.log (varbinds[i].oid + "|" + varbinds[i].value);
 		}
 	}
+	session.close();
+});
+
+session.on("error", function (error) {
+	console.error (error.toString ());
 });

package/index.js

@@ -4493,6 +4493,18 @@ Mib.prototype.deleteTableRow = function (table, rowIndex) {
 	return true;
 };
 
+Mib.prototype.getAncestorProviderFromOid = function (oid) {
+	const address = Mib.convertOidToAddress (oid);
+	for ( let i = address.length - 1 ; i >= 0 ; i-- ) {
+		const oidToCheck = address.slice(0, i).join('.');
+		const provider = this.providersByOid[oidToCheck];
+		if ( provider ) {
+			return provider;
+		}
+	}
+	return null;
+};
+
 Mib.prototype.dump = function (options) {
 	if ( ! options ) {
 		options = {};
@@ -4751,7 +4763,7 @@ Agent.prototype.castSetValue = function ( type, value ) {
 			}
 
 		case ObjectType.OID:
-			if ( typeof value != "string" || ! value.match(/[0-9]+\([.][0-9]+\)+/) ) {
+			if ( typeof value != "string" || ! value.match(/^([0-9]+)(\.[0-9]+)+$/) ) {
 				throw new Error("Invalid OID", value);
 			}
 			return value;
@@ -4993,13 +5005,24 @@ Agent.prototype.request = function (socket, requestMessage, rinfo) {
 				operation: requestPdu.type,
 				oid: requestPdu.varbinds[i].oid
 			});
-			handlers[i] = function getNsoHandler (mibRequestForNso) {
-				mibRequestForNso.done ({
-					errorStatus: ErrorStatus.NoError,
-					type: ObjectType.NoSuchObject,
-					value: null
-				});
-			};
+			const ancestorProvider = this.mib.getAncestorProviderFromOid(requestPdu.varbinds[i].oid);
+			if ( ancestorProvider ) {
+				handlers[i] = function getNsiHandler (mibRequestForNsi) {
+					mibRequestForNsi.done ({
+						errorStatus: ErrorStatus.NoError,
+						type: ObjectType.NoSuchInstance,
+						value: null
+					});
+				};
+			} else {
+				handlers[i] = function getNsoHandler (mibRequestForNso) {
+					mibRequestForNso.done ({
+						errorStatus: ErrorStatus.NoError,
+						type: ObjectType.NoSuchObject,
+						value: null
+					});
+				};
+			}
 		} else {
 			providerNode = this.mib.getProviderNodeForInstance (instanceNode);
 			if ( ! providerNode || instanceNode.value === undefined ) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "net-snmp",
-  "version": "3.15.1",
+  "version": "3.15.3",
   "description": "JavaScript implementation of the Simple Network Management Protocol (SNMP)",
   "main": "index.js",
   "directories": {
@@ -47,6 +47,6 @@
   "author": "Mark Abrahams <mark@abrahams.co.nz>",
   "license": "MIT",
   "scripts": {
-    "lint": "./node_modules/.bin/eslint . ./**/*.js"
+    "lint": "eslint . ./**/*.js"
   }
 }