github-actions[bot] opened a new pull request, #71069: URL: https://github.com/apache/airflow/pull/71069
* Mask list-shaped Variable values on deserialization _mask_and_deserialize_variable dispatched on the top-level type of the deserialized value and handled only str and dict, so a Variable whose JSON is a list was returned with no masking applied to anything inside it. add_mask walks iterables itself, so the same list nested one level inside a dict was already masked -- only a top-level one was skipped. The list is passed under the variable's key rather than anonymously: elements have no key names of their own, so they follow the variable key's sensitivity, and a list of ordinary values such as region names is not added to the global pattern set. Dicts inside the list are still masked by their own key names. * Tighten the comments around Variable masking dispatch The rationale was spread across three sites saying the same thing; keep it at the branch where the decision is made. (cherry picked from commit b968192cd3aeb55fad3bb5dc0d88917b4b29407f) Co-authored-by: Jarek Potiuk <[email protected]> -- 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]
