• src/sbbs3/con_out.cpp main.cpp mqtt.c mqtt.h putnode.cpp

    From Rob Swindell (on Windows 11)@1:103/705 to Git commit to main/sbbs/master on Sat Sep 7 15:31:15 2024
    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"?).
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)