juan-pablo-guereca commented on code in PR #68975:
URL: https://github.com/apache/airflow/pull/68975#discussion_r3629451971


##########
task-sdk/src/airflow/sdk/execution_time/context.py:
##########
@@ -317,6 +317,16 @@ async def _async_get_connection(conn_id: str) -> 
Connection:
     raise AirflowNotFoundException(f"The conn_id `{conn_id}` isn't defined")
 
 
+def _mask_and_deserialize_variable(raw: str, key: str, deserialize_json: bool) 
-> Any:
+    mask_secret(raw, key)
+    if not deserialize_json:
+        return raw
+    val = json.loads(raw)
+    if not isinstance(val, str):
+        mask_secret(val)

Review Comment:
   I've pushed a new commit with the fixes and more unit tests.



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