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

SRV connection do not work (with fix) #493

Open
edwin-oetelaar opened this issue May 22, 2020 · 5 comments
Open

SRV connection do not work (with fix) #493

edwin-oetelaar opened this issue May 22, 2020 · 5 comments
Labels
Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug

Comments

@edwin-oetelaar
Copy link

edwin-oetelaar commented May 22, 2020

return self.connect(host, port, keepalive, bind_address, clean_start, properties)

Please replace line with parameters instead of wrong positional arguments.
Testen on python 2.7.x and 3.5.x fixes both.

return self.connect(host, port, keepalive, bind_address, clean_start, properties)

            # Edwin van den Oetelaar fixes this 2020-05-22
            return self.connect(host=host, port=port, keepalive=keepalive, bind_address=bind_address, clean_start=clean_start, properties=properties)
@TD22057
Copy link

TD22057 commented Oct 5, 2020

In case this bug isn't clear - this is serious (i.e. breaking) issue. The connect line in client.py is not passing the arguments through in the correct locations to connect(). As far as I can see, this means args like clean_start don't work at all.

@edwin-oetelaar
Copy link
Author

I am not sure how to contribute here even more than I already did.
My code running a real project broke, I needed the SRV functions.
I found the problem, debugged it, fixed it and submitted a patch.
Who is responsible for maintaining this code base?

Also #442 (comment)

@TD22057
Copy link

TD22057 commented Oct 31, 2020

Any update on this? This bug breaks the client and is trivial to fix.

@and3rson
Copy link

and3rson commented Feb 1, 2021

Any updates? This really breaks the connect_srv and makes it unusable.

@edwin-oetelaar
Copy link
Author

return self.connect(host, port, keepalive, bind_address, clean_start, properties)

this looks still not fixed

@icraggs : please fix what you broke

mijofa added a commit to mijofa/paho.mqtt.python that referenced this issue Oct 23, 2023
@MattBrittan MattBrittan added Type: Bug Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. labels Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug
Projects
None yet
Development

No branches or pull requests

4 participants