Skip to content

Conversation

peyoot
Copy link

@peyoot peyoot commented Sep 20, 2022

add additonal MQTT_TLS flag and CA, cert, key file in settings.py
tested mqtts support and it works fine!

@lbt
Copy link
Owner

lbt commented Nov 21, 2022

In principle these changes are acceptable. However they cannot be merged in this PR.

Please update the README to add information to this upstream rather that as if you were forking it.

The settings for MQTT_TLS should handle the setting being undefined.
The mqtt.connect() should handle SSL and non-SSL
Don't change the author/email/packages in setup.cfg

Thanks

(Also apologies for the slow response - this project's emails were going into my low-priority folder)

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