Merge pull request #1509 from chakflying/feat/mqtt-optional-message

Feat: Allow MQTT successMessage to be optional

server/server.js

@@ -721,6 +721,7 @@ try {
                 bean.pushToken = monitor.pushToken;
                 bean.proxyId = Number.isInteger(monitor.proxyId) ? monitor.proxyId : null;
                 bean.mqttUsername = monitor.mqttUsername;
+                bean.mqttPassword = monitor.mqttPassword;
                 bean.mqttTopic = monitor.mqttTopic;
                 bean.mqttSuccessMessage = monitor.mqttSuccessMessage;
 

server/util-server.js

@@ -135,10 +135,10 @@ exports.mqttAsync = function (hostname, topic, okMessage, options = {}) {
             if (messageTopic == topic) {
                 client.end();
                 clearTimeout(timeoutID);
-                if (message.toString() === okMessage) {
+                if (okMessage != null && okMessage !== "" && message.toString() !== okMessage) {
-                    resolve(`Topic: ${messageTopic}; Message: ${message.toString()}`);
+                    reject(new Error(`Message Mismatch - Topic: ${messageTopic}; Message: ${message.toString()}`));
                 } else {
-                    reject(new Error(`Error; Topic: ${messageTopic}; Message: ${message.toString()}`));
+                    resolve(`Topic: ${messageTopic}; Message: ${message.toString()}`);
                 }
             }
         });

src/pages/EditMonitor.vue

@@ -141,7 +141,7 @@
 
                                 <div class="my-3">
                                     <label for="mqttSuccessMessage" class="form-label">MQTT {{ $t("successMessage") }}</label>
-                                    <input id="mqttSuccessMessage" v-model="monitor.mqttSuccessMessage" type="text" class="form-control" required>
+                                    <input id="mqttSuccessMessage" v-model="monitor.mqttSuccessMessage" type="text" class="form-control">
                                     <div class="form-text">
                                         {{ $t("successMessageExplanation") }}
                                     </div>