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

Fix 'asyncio_mqtt' has no attribute 'TLSParameters' error #129

Merged
merged 1 commit into from
Oct 6, 2022
Merged

Fix 'asyncio_mqtt' has no attribute 'TLSParameters' error #129

merged 1 commit into from
Oct 6, 2022

Conversation

empicano
Copy link
Owner

@empicano empicano commented Oct 5, 2022

Hi Frederik 😊

Using the current code straight from GitHub leads to a 'asyncio_mqtt' has no attribute 'TLSParameters' error on setting TLS configuration. I think this is because the import in __init__.py is missing. #126 is the relevant commit.

In case you're looking for help, I'll try to set some time apart to help out, as I'm actively using asyncio-mqtt at the moment.

@frederikaalund
Copy link
Collaborator

Hi Felix, thanks for opening this PR. Let me have a look. :)

Ah, nice find! This is an oversight on my part. Thanks for the fix. 👍

In case you're looking for help, I'll try to set some time apart to help out, as I'm actively using asyncio-mqtt at the moment.

Greatly appreciated. :) Feel free to help out wherever you deem necessary. I'll try to review issues/PRs/discussions in a timely fashion. If I don't respond within a day, just ping me. 👍

@frederikaalund frederikaalund merged commit 912dd73 into empicano:master Oct 6, 2022
@empicano empicano deleted the fix-missing-tlsparameter-export branch October 6, 2022 12:39
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

Successfully merging this pull request may close these issues.

2 participants