dstandish commented on code in PR #31820:
URL: https://github.com/apache/airflow/pull/31820#discussion_r1224753581
##########
tests/core/test_configuration.py:
##########
@@ -1447,3 +1448,36 @@ def test_future_warning_only_for_code_ref(self, key):
w = captured.pop()
assert "your `conf.get*` call to use the new name" in
str(w.message)
assert w.category == FutureWarning
+
+
+def test_sensitive_values():
+ from airflow.settings import conf
+
+ # this list was hardcoded prior to 2.6.2
+ # included here to avoid regression in refactor
+ # inclusion of keys ending in "password" or "kwargs" is automated from
2.6.2
Review Comment:
it bit us recently. there are scenarios when users may supply sensitive
info this way. not a good practice but it happens apparently.
--
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]
