Undefined node and options in addMsg

[Beetix]

Hi,

There is a bug when using TLS. node and options variables are not defined in the addMsg function:

IgnitionNode-RED/node-red-contrib-ignition/node-handlers.js

Lines 194 to 199 in 3a52c20

	if (this.serverConfig.ssl && this.serverConfig.tls) {
	var tlsNode = RED.nodes.getNode(node.serverConfig.tls);
	if (tlsNode) {
	tlsNode.addTLSOptions(options);
	}
	}

It's seems to be a copy/paste from startconn:

IgnitionNode-RED/node-red-contrib-ignition/node-handlers.js

Lines 395 to 405 in 3a52c20

	var options = {};
	if (agent) {
	options.agent = agent;
	}
	if (node.serverConfig.ssl && node.serverConfig.tls) {
	var tlsNode = RED.nodes.getNode(node.serverConfig.tls);
	if (tlsNode) {
	tlsNode.addTLSOptions(options);
	}
	}

Removing lines 194-199 in node-handlers.js fixes the problem. If I understand correctly this code is useful when using a proxy, right?

Thanks

[iatraviscox]

That was a copy paste issue. I have released a new version with the fix.

[Beetix]

Great! Thank you for fixing that so quickly!