NewSmoke38 commented on PR #58659:
URL: https://github.com/apache/airflow/pull/58659#issuecomment-3698243519
Hi @arnoldmr01, @potiuk, and @jason810496!
I’ve prepared the documentation updates for set-config.rst and the
docker-compose.yaml to align them with the new masking logic. Since I can't
push or suggest in the main branch as these files aren't in the PR yet, I am
putting my changes here:
1. In `set-config.rst`
Line 31:
```
airflow config list --defaults --show-values
```
Line 45:
```
airflow config list --defaults --show-values >
"${AIRFLOW_HOME}/airflow.cfg"
```
after this add this note -
```
.. note::
By default, Airflow masks sensitive values (passwords, secrets,
connection strings)
in CLI output to prevent accidental exposure. You must include the
``--show-values`` flag when generating or
exporting configuration files.
Running ``airflow config list`` without this flag and redirecting to a
file will save non-functional placeholders
(e.g., ``<hidden>``). This will cause Airflow to fail when it attempts
to use those configurations.
```
2. In `docker-compose.yaml`
Line 270:
```
/entrypoint airflow config list --show-values >/dev/null
```
I'll search across the entire codebase now to identify any other instances
in documentation or scripts where `airflow config list` is used for
initialization or export. I'll follow up here if I find more areas that need
updating.
--
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]
