potiuk commented on code in PR #58659:
URL: https://github.com/apache/airflow/pull/58659#discussion_r2583961037
##########
airflow-core/src/airflow/cli/cli_config.py:
##########
@@ -789,6 +789,11 @@ def string_lower_type(val):
("option",),
help="The option name",
)
+ARG_CONFIG_HIDE_SENSITIVE = Arg(
+ ("--hide-sensitive",),
+ action="store_true",
+ help="Hide sensitive configuration values (passwords, keys, etc.) in the
output.",
+)
Review Comment:
We agreed on the devlist (and there was a lazy consensus on that) - see
https://lists.apache.org/thread/3dhzqvpw4z2x0wfokpmdncsjwws86zcs point 5:
> 5) local CLI * list (connections, variables, config) only by default
returns "keys" - and it will only return values when `--show-values`
is passed as command line option (with clear comment in help that this
option **might** show sensitive data, also when we do `* list` command
without `--show-values` we emit stderr output explaining that
potentially sensitive data is hidden and you need to specify
`--show-values` to see them
--
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]
