bujjibabukatta opened a new pull request, #70883:
URL: https://github.com/apache/airflow/pull/70883

   **Summary**
   Deprecation warning for airflow.sdk.execution_time.secrets_masker tells
   everyone to switch to airflow.sdk._shared.secrets_masker — but for
   mask_secret specifically that's wrong and silently weakens masking if
   followed.
   
   **Root cause**
   __getattr__ deliberately resolves mask_secret from airflow.sdk.log instead
   of _shared, since airflow.sdk.log.mask_secret also notifies the supervisor
   process while the _shared version doesn't. The warning text never carved
   out this exception.
   
   **Fix**
   Updated the warning and docstring to name airflow.sdk.log for mask_secret.
   __getattr__ itself is unchanged. Added tests asserting object identity and
   warning-message content.
   
   closes: #70875
   
   Was generative AI tooling used ?
   
   - [X] Yes - Claude
   
   Generated-by: Claude following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


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