MQTT node should only send message if topic actually specified (string)

2025-03-01 10:36:34 +00:00 · 2014-09-08 20:01:28 +01:00
parent 1a5a548fbf
commit ab4d3c025e
1 changed files with 4 additions and 1 deletions
--- a/nodes/core/io/10-mqtt.js
+++ b/nodes/core/io/10-mqtt.js
@@ -95,7 +95,10 @@ module.exports = function(RED) {
                    if (node.topic) {
                        msg.topic = node.topic;
                    }
-                    this.client.publish(msg);
+                    if (typeof msg.topic === "string") { // topic must be a string
+                        this.client.publish(msg);  // send the message
+                    }
+                    else { node.warn("Invalid topic specified"); }
                }
            });
            this.client.on("connectionlost",function() {