Apply suggestions from code review

Co-authored-by: Frank Elsinga <frank@elsinga.de>

server/model/monitor.js

@@ -602,9 +602,12 @@ class Monitor extends BeanModel {
 
                         const { status, response } = await evaluateJsonQuery(data, this.jsonPath, this.jsonPathOperator, this.expectedValue);
 
-                        bean.status = status ? UP : DOWN;
+                        if (status) {
-                        bean.msg = `JSON query ${status ? "passes" : "does not pass"} `;
+                            bean.status = UP; 
-                        bean.msg += `comparison: ${response} ${this.jsonPathOperator} ${this.expectedValue}.`;
+                            bean.msg = `JSON query passes (comparing ${response} ${this.jsonPathOperator} ${this.expectedValue})`
+                        } else {
+                            throw new Error(`JSON query does not pass (comparing ${response} ${this.jsonPathOperator} ${this.expectedValue})`);
+                        }
 
                     }
 

server/monitor-types/snmp.js

@@ -9,17 +9,15 @@ class SNMPMonitorType extends MonitorType {
      * @inheritdoc
      */
     async check(monitor, heartbeat, _server) {
-
+        let session;
-        const options = {
+        try {
+            const sessionOptions = {
                 port: monitor.port || "161",
                 retries: monitor.maxretries,
                 timeout: monitor.timeout * 1000,
                 version: snmp.Version[monitor.snmpVersion],
             };
-
+            session = snmp.createSession(monitor.hostname, monitor.radiusPassword, sessionOptions);
-        let session;
-        try {
-            session = snmp.createSession(monitor.hostname, monitor.radiusPassword, options);
 
             // Handle errors during session creation
             session.on("error", (error) => {
@@ -46,13 +44,14 @@ class SNMPMonitorType extends MonitorType {
 
             const { status, response } = await evaluateJsonQuery(value, monitor.jsonPath, monitor.jsonPathOperator, monitor.expectedValue);
 
-            heartbeat.status = status ? UP : DOWN;
+            if (status) {
-            heartbeat.msg = `SNMP value ${status ? "passes" : "does not pass"} `;
+                bean.status = UP; 
-            heartbeat.msg += `comparison: ${response} ${monitor.jsonPathOperator} ${monitor.expectedValue}.`;
+                bean.msg = `JSON query passes (comparing ${response} ${this.jsonPathOperator} ${this.expectedValue})`
-
+            } else {
+                throw new Error(`JSON query does not pass (comparing ${response} ${this.jsonPathOperator} ${this.expectedValue})`);
+            }
         } catch (err) {
-            heartbeat.status = DOWN;
+            throw err;
-            heartbeat.msg = `Error: ${err.message}`;
         } finally {
             if (session) {
                 session.close();

src/pages/EditMonitor.vue

@@ -1354,7 +1354,7 @@ message HealthCheckResponse {
 
             // Set default SNMP version
             if (!this.monitor.snmpVersion) {
-                this.monitor.snmpVersion = "1";
+                this.monitor.snmpVersion = "2c";
             }
 
             // Set default jsonPath