Subj : Add a connect callback handler.
To   : GitLab note in main/sbbs
From : Rick Parrish
Date : Mon Aug 21 2023 03:55 pm

https://gitlab.synchro.net/main/sbbs/-/merge_requests/320#note_3833

Ahh, bad assumption on my part then!

So in hindsight, maybe these two lines should have stayed in mqtt_startup, since it's only the subscriptions that need to be re-applied in the connect callback when a reconnect occurs:

mqtt_pub_noval(mqtt, TOPIC_SERVER, "recycle");
mqtt_pub_noval(mqtt, TOPIC_SERVER, "client");

I just tested that here, and my Linux build no longer hits a recycle loop even with the Publish QOS set to 1.

.