Make connect() non-blocking

[oholsen]

Resolves #22.

[frederikaalund]

Hi Øystein, thanks for the pull request. Let me have a look. :)

I did not realize that paho.Client.connect is blocking. Good catch!

I had a brief look at paho.Client.connect_async that sounds promising but unfortunately doesn't work for our use case. At least not without reimplementing a significant portion of paho.mqtt ourselves.

In any case, seems to me like it's the right approach to simply execute paho.Client.connect_async in another thread (as done in your pull request).

Good to see a test as well. 👍 We need more pull requests in that area.

Thanks for you contribution. If you want, you can add your change to the changelog and attribute it to yourself.

[oholsen]

@frederikaalund Will there be a release soon that incorporates this and other fixes?

[frederikaalund]

I plan to close some of the issues this week. Afterwards, I'll make a release. :)

[oholsen]

Great! 😀

…

On Tue, 16 Feb 2021, 10:35 Frederik Aalund, ***@***.***> wrote: I plan to close some of the issues this week. Afterwards, I'll make a release. :) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#23 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABT3SE2JAB2OCOXXFTM2WVLS7I37DANCNFSM4TQLYW7Q> .

[frederikaalund]

I just released v0.8.1 👍

Sorry about the delay.

[oholsen]

Thanks! 😀

…

On Tue, 23 Feb 2021, 10:53 Frederik Aalund, ***@***.***> wrote: I just released v0.8.1 👍 Sorry about the delay. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#23 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABT3SEZRUYW6FWU6R6VSUMDTAN3LLANCNFSM4TQLYW7Q> .