Fix #14973: Qliksense - Add option to disable hostname validation (#15954)

ingestion/src/metadata/ingestion/source/dashboard/qliksense/client.py

@@ -73,6 +73,7 @@ class QlikSenseClient:
                 "ca_certs": self.config.certificates.rootCertificate,
                 "certfile": self.config.certificates.clientCertificate,
                 "keyfile": self.config.certificates.clientKeyCertificate,
+                "check_hostname": self.config.validateHostName,
             }
             return context
 

ingestion/src/metadata/utils/ssl_manager.py

@@ -124,11 +124,12 @@ class SSLManager:
         return connection
 
     @setup_ssl.register(QlikSenseConnection)
-    def _(self, _):
+    def _(self, connection):
         return {
             "ca_certs": self.ca_file_path,
             "certfile": self.cert_file_path,
             "keyfile": self.key_file_path,
+            "check_hostname": connection.validateHostName,
         }
 
     @setup_ssl.register(KafkaConnection)

openmetadata-docs/content/v1.4.x-SNAPSHOT/connectors/dashboard/qliksense/index.md

@@ -82,6 +82,8 @@ You will have to replace new lines with `\n` and the final private key that you
 
 **User ID**: This field specifies the user id of the user.
 
+**Validate Host Name**: Enable/Disable this field to validate the host name against the provided certificates.
+
 {% /extraContent %}
 
 {% partial file="/v1.4/connectors/test-connection.md" /%}

openmetadata-docs/content/v1.4.x-SNAPSHOT/connectors/dashboard/qliksense/yaml.md

@@ -130,6 +130,12 @@ You will have to replace new lines with `\n` and the final private key that you
 
 {% /codeInfo %}
 
+{% codeInfo srNumber=7 %}
+
+**validateHostName**: Enable/Disable this field to validate the host name against the provided certificates.
+
+{% /codeInfo %}
+
 {% partial file="/v1.4/connectors/yaml/dashboard/source-config-def.md" /%}
 
 {% partial file="/v1.4/connectors/yaml/ingestion-sink-def.md" /%}
@@ -174,7 +180,9 @@ source:
 ```yaml {% srNumber=6 %}
       userDirectory: user_dir
 ```
-
+```yaml {% srNumber=6 %}
+      validateHostName: false
+```
 {% partial file="/v1.4/connectors/yaml/dashboard/source-config.md" /%}
 
 {% partial file="/v1.4/connectors/yaml/ingestion-sink.md" /%}

openmetadata-spec/src/main/resources/json/schema/entity/services/connections/dashboard/qlikSenseConnection.json

@@ -86,6 +86,12 @@
                 }
             ]
         },
+        "validateHostName": {
+            "title": "Validate Host Name",
+            "description": "Validate Host Name",
+            "type": "boolean",
+            "default": false
+        },
         "userDirectory": {
             "title": "User Directory",
             "description": "User Directory.",

openmetadata-ui/src/main/resources/ui/public/locales/en-US/Dashboard/QlikSense.md

@@ -153,4 +153,11 @@ $$section
 ### User ID $(id="userId")
 
 This field specifies the user id of the user.
+$$
+
+
+$$section
+### Validate Host Name $(id="validateHostName")
+
+Enable/Disable this field to validate the host name against the provided certificates.
 $$