You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/05/17 09:10:36 UTC

[GitHub] [airflow] potiuk commented on a diff in pull request #31171: Introduce configuration to disable md5 used for security

potiuk commented on code in PR #31171:
URL: https://github.com/apache/airflow/pull/31171#discussion_r1196188099


##########
airflow/cli/commands/webserver_command.py:
##########
@@ -40,6 +39,11 @@
 from airflow.exceptions import AirflowException, AirflowWebServerTimeout
 from airflow.utils import cli as cli_utils
 from airflow.utils.cli import setup_locations, setup_logging
+
+try:
+    from airflow.utils.hashlib_wrapper import md5
+except ModuleNotFoundError:
+    from hashlib import md5

Review Comment:
   NIT: we do not need it here, try/except is only needed in providers.



-- 
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