amoghrajesh commented on code in PR #54449:
URL: https://github.com/apache/airflow/pull/54449#discussion_r2293171348
##########
airflow-core/src/airflow/dag_processing/processor.py:
##########
@@ -409,6 +409,10 @@ def _handle_request(self, msg: ToManager, log:
FilteringBoundLogger, req_id: int
elif isinstance(msg, GetVariable):
var = self.client.variables.get(msg.key)
if isinstance(var, VariableResponse):
+ if var.value:
+ from airflow.sdk.log import mask_secret
+
+ mask_secret(var.value, var.key)
Review Comment:
Good catch, yeah it is already handled for dag processor and works without
needing this, however, it doesnt work as expected for triggerer if I remove it.
I have left a TODO for it, and that can be handled as an immediate follow-up.
Or if you know about it we could collab! I tried debugging a bit and couldnt
get to the bottom of it
--
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]
