andylamp commented on code in PR #32867:
URL: https://github.com/apache/airflow/pull/32867#discussion_r1275464755
##########
airflow/models/connection.py:
##########
@@ -43,6 +45,23 @@ def parse_netloc_to_hostname(*args, **kwargs):
return _parse_netloc_to_hostname(*args, **kwargs)
+def sanitize_conn_id(conn_id: str | None) -> str | None:
Review Comment:
it's going to be the default behaviour in case a malformed one is
encountered. Happy to raise an exception, if that's preferred but did not want
to raise an exception in an unexpected location. Any suggestions?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
