arnoldmr01 commented on code in PR #58659:
URL: https://github.com/apache/airflow/pull/58659#discussion_r2646289668
##########
airflow-core/src/airflow/cli/cli_config.py:
##########
@@ -790,6 +790,20 @@ def string_lower_type(val):
("option",),
help="The option name",
)
+ARG_NON_SENSITIVE = Arg(
+ ("--non-sensitive",),
+ action="store_true",
+ help="When used with --show-values, hide sensitive values (passwords,
keys, tokens, etc.) and only show non-sensitive configuration values.",
+)
+ARG_CONFIG_SHOW_VALUES = Arg(
+ ("--show-values",),
+ action="store_true",
+ help=(
+ "Show configuration values. "
+ "By default only option names are shown and values (including
potentially "
+ "sensitive ones) are hidden."
+ ),
+)
Review Comment:
Thanks for the suggestion. While it does preserve the current behavior, the
dev list here (https://lists.apache.org/thread/3dhzqvpw4z2x0wfokpmdncsjwws86zcs
) agreed that * list should return keys only; values should be shown via
--show-values.
--
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]
