github
/
apprise
mirror of https://github.com/caronc/apprise
1
0
Fork 0
Code Issues Releases Wiki Activity

fix insecure SSL check on mqtts:// configurations (#825)

pull/832/head
Chris Caron 2023-02-18 20:21:19 -05:00 committed by GitHub
parent 58f9e64843
commit 7bcb0647c0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
apprise/plugins/NotifyMQTT.py
View File

@ -324,7 +324,7 @@ class NotifyMQTT(NotifyBase):
ciphers=None)
# Set our TLS Verify Flag
self.client.tls_insecure_set(self.verify_certificate)
self.client.tls_insecure_set(not self.verify_certificate)
# Establish our connection
if self.client.connect(

4
test/test_plugin_mqtt.py
View File

@ -252,7 +252,7 @@ def test_plugin_mqtt_tls_connect_success(mqtt_client_mock):
tls_version=ssl.PROTOCOL_TLS,
ciphers=None,
),
call.tls_insecure_set(True),
call.tls_insecure_set(False),
call.connect('localhost', port=8883, keepalive=30),
call.loop_start(),
call.is_connected(),
@ -299,7 +299,7 @@ def test_plugin_mqtt_tls_no_verify_success(mqtt_client_mock):
# Verify the right calls have been made to the MQTT client object.
# Let's only validate the single call of interest is present.
# Everything else is identical with `test_plugin_mqtt_tls_connect_success`.
assert call.tls_insecure_set(False) in mqtt_client_mock.mock_calls
assert call.tls_insecure_set(True) in mqtt_client_mock.mock_calls
def test_plugin_mqtt_session_client_id_success(mqtt_client_mock):