Subj : src/sbbs3/con_out.cpp main.cpp mqtt.c mqtt.h putnode.cpp
To   : Git commit to main/sbbs/master
From : Rob Swindell (on Windows 11)
Date : Sat Sep 07 2024 03:31 pm

https://gitlab.synchro.net/main/sbbs/-/commit/9b6138f2b579e90cf7e9dd97
Modified Files:
	src/sbbs3/con_out.cpp main.cpp mqtt.c mqtt.h putnode.cpp
Log Message:
Don't attempt to publish MQTT messages unless/until connected to broker

For cases where an mqtt struct is shared between threads without concurrency
control. I'm making this improvement in light of research into issue #781,
though I don't expect this change to fix the reported issue.

The reported error seems to come from the event thread (publishing node
status upon starting to run the "DAILY" event) when a broker connection was
not successful, however the reporter (Nelgin) may not have had debug-level
logging turned on, so didn't capture the successful broker-connect log
message. I think the broker connection *was* successful and perhaps then
terminated by the broker ("due to protocol error"?).

.