micropython-lib icon indicating copy to clipboard operation
micropython-lib copied to clipboard

Umqtt.robust: subscription vanished after reconnect

Open JDchauhan opened this issue 7 years ago • 12 comments

Board used ESP8266

What is the problem: Umqtt.robust() on reconnection will forget the previously subscribed topics i.e, it do not fetch messages of subscribed topics after reconnection.

When the problem occurs (how to mimic it) after successful subscription of topics and verifying that everything works fine just turn off your internet connection. Then after 5 minutes restart internet connection.

(Note: try to use publish-message with try-catch before check_msg() otherwise u might get stuck with #192 )

Then you will observe that no message is being fetched by the check_msg() on that topic.

How to solve this This might not be a good solution but for now, I am just subscribing all the topics again whenever the connection lost

JDchauhan avatar Nov 27 '18 10:11 JDchauhan

That's the only working solution and the best one. I'm doing so as well with another mqtt client. The only other solution would be to connect without a clean connect but that can result in getting all missed messages between connection loss and reconnection and is therefore not advised to do as it would likely crash your small microcontroller.

kevinkk525 avatar Dec 03 '18 16:12 kevinkk525

How to you re-subscribe to all topics on connection lost??

turutupa avatar Feb 11 '20 11:02 turutupa

You have all your subscriptions stored and on reconnect you just subscribe all of them.

kevinkk525 avatar Feb 11 '20 11:02 kevinkk525

But I mean. If on my main while loop I only have client.wait_msg(). Where exactly do I execute the re-subscribe? Is there an onconnect / onreconnect function or something like this? (thanks for the quick response btw!)

turutupa avatar Feb 11 '20 11:02 turutupa

Hmm you are right, the umqtt.robust doesn't really offer a convenient way of recognizing a reconnect.. I haven't used this code in years. I use an async mqtt client https://github.com/peterhinch/micropython-mqtt

It looks like you need to create a subclass so you can change the way the reconnect works so you can subscribe after a reconnect. But maybe someone else who actually uses this code has a better solution.

kevinkk525 avatar Feb 11 '20 15:02 kevinkk525

How to you re-subscribe to all topics on connection lost??

I have maintained list of subscribed topic in my code itself, Whenever It gots to know about disconnection and gets reconnected I will again subscribing them, for prompt of disconnect I am running the publish event (just for checking) in the try block and checking problems in catch if any comes

JDchauhan avatar Feb 12 '20 10:02 JDchauhan

UUuuuh, nice. Hadn´t thought about publishing just to check if it is subscribed to a topic, nice approach! Mainly cause I have only used umqtt to receive messages.

Now I fully understand your note

try to use publish-message with try-catch before check_msg() otherwise u might get stuck with #192 )

Thanks!

turutupa avatar Feb 12 '20 11:02 turutupa

Hey guys, I am back and not with good news hehe.. I was wondering, @JDchauhan could you provide code to your publish-subscribe? I haven't spent too much time on this, but I must admit I haven't managed to make it work. I have tried to try-catch publish and then check_msg but it gives me mqtt: OSError(-1,)

Hoping to find a robust way to make it work

IMPORTANT EDIT: forgot to mention. If I only publish every 5 ~ 10 seconds its fine. It does publish. Gives me error when I check_msg() after publishing

I have to add I am using Adafruit IO to publish-subscribe... could that be issue? What service are you using? Perhaps I should run my own mosquitto on a pi? I rather use Adafruit for IFTTT...

turutupa avatar Feb 27 '20 11:02 turutupa

Hey guys, I am back and not with good news hehe.. I was wondering, @JDchauhan could you provide code to your publish-subscribe? I haven't spent too much time on this, but I must admit I haven't managed to make it work. I have tried to try-catch publish and then check_msg but it gives me mqtt: OSError(-1,)

Hoping to find a robust way to make it work

IMPORTANT EDIT: forgot to mention. If I only publish every 5 ~ 10 seconds its fine. It does publish. Gives me error when I check_msg() after publishing

I have to add I am using Adafruit IO to publish-subscribe... could that be issue? What service are you using? Perhaps I should run my own mosquitto on a pi? I rather use Adafruit for IFTTT...

Seeing the exact same issue, with Adafruit IO too.

alessionossa avatar Jun 14 '23 13:06 alessionossa

Hi @turutupa & @alessionossa I was using my own MQTT broker at that time.

Below is the part of the code I used back then that worked for me. I don't remember exactly but from the code, it seems like I am expecting the error to be thrown by check_msg() instead of publish() and then subscribing again (if it helps)

            try:
                client.publish(topic="test", msg="testing")
                if not isBroker:
                    client.subscribe(topic= data["id"] + "/#")                     
                # print("connected")
                client.check_msg()
                time.sleep(0.5)
            except:
                isBroker = False
                print("broker disconnected")
                time.sleep(2)
                break

JDchauhan avatar Jul 22 '23 18:07 JDchauhan

Same on my side. I'm using micropython on ESP32 C3. Next the workaround that works for me. Main loop blocking: In the main() loop, I've replaced the mqtt.robust check_msg() and wait_msg() by my own, because if there is no mqtt server, check_msg() is waiting forever. That happens because when it calls to wait_msg(), it waits forever instead of generate an exception that try to reconnect and then continue the main() loop The problem arise because check_msg() set self.mqtt_client.sock.setblocking(False), and then calls to wait_msg() On their side wait_msg() change self.mqtt_client.sock.setblocking(True) at its beginning, and then allowing to wait for a mqtt server connection. My workaround is using my_check_msg(), that is an exact copy of the check_msg() except it calls my_wait_msg(). my_wait_msg() is an exact copy of wait_msg() except that the initial #self.mqtt_client.sock.setblocking(True) is commented. I've tried this with class inheritance and monkey patching without success. May be side effects Regenerate subscription mqtt: All my appliances (sensor, switch, light,...) connected to mqtt have a _register_mqtt(self) procedure. In the initial setup of every appliance, I add each appliance to a device_appliances{} dictionary (key=appl_id, value=appliance) and also call _register_mqtt(self) To reconnect after a mqtt server (or Wifi) fault, the procedure reinitialize the mqtt_client and then calls the _register_mqtt() for each appliance in the device_appliances{} dictionary. Finally also republish the availability_topic.

javiergmarcos avatar Aug 12 '23 18:08 javiergmarcos

Subclass is the current recommended solution here, however it won't really work until this is merged. Take a look for more information & example about re-subscription https://github.com/micropython/micropython-lib/pull/669

andrewleech avatar Aug 12 '23 22:08 andrewleech