-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
MQTTS Verify=No does not ignore self-signed certificates #802
Comments
This is a good find on your part. It turns out the effects were actually reversed. In your case, setting Can you test the PR (instructions are detailed inside it) and let me know if this fixes your issue? |
I tested it and it appears to work for me, so I'm closing this ticket now. |
Hi @caronc - Sorry for the delay on getting back to this! Unfortunately with the update the issue still persists. There HAS been a change in behaviour, as setting (I am running this using https://github.com/dgtlmoon/changedetection.io, which is using the up-to-date version of Apprise). Any further thoughts? |
Unfortunately I've just came here from the Tube Archivist, and I'm having the same problem with version 1.9.0 (after installing paho-mqtt v1.6.1 manually the container doesn't ship with it).
|
Do you know what version of mqtt you're using? Perhaps it's related to an older, (or newer) version on the system you're using? |
My MQTT broker, Mosquitto 2.0.11, can do MQTT v5, v3.11 and v3.1. |
Which Python version are you using? The SSL library changed how it behaves in 3.4 and 3.7, but I can't see anything in the new paho-mqtt specifically that affects the way SSL works. Maybe apprise should not set |
|
Here's the story: The original way of doing this in Many months later, the developers of # Current paho_mqtt documented way of doing this
pahomqtt.obj.tls_insecure_set(True) Best advice would be to use The best i can do is go back to pinning an old version of the dependency again (and drop > v2 support once more) |
📣 Notification Service(s) Impacted
🐞 Describe the bug
💡 Screenshots and Logs
![image](https://user-images.githubusercontent.com/75918408/213568456-52aa593c-78be-4f15-88b2-2bf22c54519a.png)
💻 Your System Details:
🔮 Additional context
?verify="/datastore/server.crt"
(with correct permissions), however this just defaults to?verify=No
in the debug log.The text was updated successfully, but these errors were encountered: