[GitHub] [airflow] c2zwdjnlcg commented on a diff in pull request #26246: trino: Support CertificateAuthentication in the trino hook

[GitBox <gi...@apache.org>]

c2zwdjnlcg commented on code in PR #26246:
URL: https://github.com/apache/airflow/pull/26246#discussion_r968550785


##########
airflow/providers/trino/hooks/trino.py:
##########
@@ -99,12 +99,19 @@ def get_conn(self) -> Connection:
         extra = db.extra_dejson
         auth = None
         user = db.login
-        if db.password and extra.get('auth') == 'kerberos':
-            raise AirflowException("Kerberos authorization doesn't support password.")
+        if db.password and extra.get('auth') in ('kerberos', 'certs'):
+            raise AirflowException(
+                "The '{}' authorization type doesn't support password.".format(extra.get('auth'))
+            )

Review Comment:
   Thanks @uranusjr, updated. Can you approve the tests workflow run



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org