Merge pull request #462 from SAIC-iSmart-API/fix/tls-insecure-requires-custom-ca

fix(tls): warn on hostname-check bypass regardless of custom CA cert

Author: nanomad

src/publisher/mqtt_publisher.py

@@ -70,10 +70,10 @@ async def __run_loop(self) -> None:
                 ssl_context.load_verify_locations(
                     cafile=self.configuration.tls_server_cert_path
                 )
-                if not self.configuration.tls_server_cert_check_hostname:
-                    LOG.warning(
-                        f"Skipping hostname check for TLS connection to {self.host}"
-                    )
+            if not self.configuration.tls_server_cert_check_hostname:
+                LOG.warning(
+                    f"Skipping hostname check for TLS connection to {self.host}"
+                )
 
         client = aiomqtt.Client(
             hostname=self.host,