mirror of
				https://github.com/node-red/node-red.git
				synced 2025-03-01 10:36:34 +00:00 
			
		
		
		
	Merge pull request #3869 from Steve-Mcl/fix-mqtt-birth-bad-topic-crash
Prevent invalid mqtt birth topic crashing node-red
This commit is contained in:
		| @@ -997,14 +997,21 @@ module.exports = function(RED) { | ||||
|                 } | ||||
|  | ||||
|                 if (topicOK) { | ||||
|                     node.client.publish(msg.topic, msg.payload, options, function(err) { | ||||
|                         done && done(err); | ||||
|                         return | ||||
|                     }); | ||||
|                     node.client.publish(msg.topic, msg.payload, options, function (err) { | ||||
|                         if (done) { | ||||
|                             done(err) | ||||
|                         } else { | ||||
|                             node.error(err, msg) | ||||
|                         } | ||||
|                     }) | ||||
|                 } else { | ||||
|                     const error = new Error(RED._("mqtt.errors.invalid-topic")); | ||||
|                     error.warn = true; | ||||
|                     done(error); | ||||
|                     const error = new Error(RED._("mqtt.errors.invalid-topic")) | ||||
|                     error.warn = true | ||||
|                     if (done) { | ||||
|                         done(error) | ||||
|                     } else { | ||||
|                         node.warn(error, msg) | ||||
|                     } | ||||
|                 } | ||||
|             } | ||||
|         }; | ||||
|   | ||||
		Reference in New Issue
	
	Block a user