ashb commented on code in PR #54449:
URL: https://github.com/apache/airflow/pull/54449#discussion_r2284570106


##########
airflow-core/src/airflow/config_templates/airflow_local_settings.py:
##########
@@ -71,22 +71,25 @@
         },
     },
     "filters": {
-        "mask_secrets": {
-            "()": "airflow.sdk.execution_time.secrets_masker.SecretsMasker",
+        "mask_secrets_core": {
+            "()": 
"airflow._shared.secrets_masker.secrets_masker._secrets_masker",
+        },
+        "mask_secrets_sdk": {
+            "()": 
"airflow.sdk._shared.secrets_masker.secrets_masker._secrets_masker",

Review Comment:
   Note to self: This is fine for this PR, but we're going to have to do 
something about this for 3.1, else this will fail to import the class if 
task-sdk isn't installed. I should address this as part of #52651



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

Reply via email to