Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disconnect after a certain number of messages were published #75

Closed
stijn-devriendt opened this issue Aug 19, 2021 · 4 comments
Closed

Comments

@stijn-devriendt
Copy link

stijn-devriendt commented Aug 19, 2021

I'd like to bring the following issue to your attention:
eclipse/paho.mqtt.python#563
I added a comment explaining the issue and how asyncio-mqtt is involved.

As far as I know, the root cause is in paho-mqtt, but is exacerbated by how asyncio-mqtt uses paho-mqtt.
I'm not sure there's much you can do about it in asyncio-mqtt, except for calling loop() instead.
However, I believe that is not a full solution since the sockpair will still get out of sync.

Most likely when paho-mqtt is fixed, you'll want to bump the required version.

@frederikaalund
Copy link
Member

Hi Stijn, thanks for bringing this issue to our attention. 👍 Like you mention yourself, this is something that paho-mqtt must fix. We can't really do that much from the asyncio-mqtt side, unfortunately. In any case, it's good to have the issue tracked here as well. :) Let us know if there are any major updates on it.

@stijn-devriendt
Copy link
Author

Hi Frederik, As you can read on the paho-mqtt issue, their 1.6.x branch contains a fix. The maintainer is aiming for a release around the end of September, so I guess that might be a good time to revisit the paho-mqtt project for an update.

@stijn-devriendt
Copy link
Author

The dependency of paho-mqtt has been updated already for another issue.
I guess this one can be closed?

@frederikaalund
Copy link
Member

You're right. Thanks for following up on this issue. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants