bugraoz93 commented on PR #61302: URL: https://github.com/apache/airflow/pull/61302#issuecomment-3831975146
> > Changes looks good but why do we need to make this error. This is purely created for integration tests and never ever should be used in any production. We should always use keyring in production. This could break integration tests because as this is warning to also users not an error > > @bugraoz93, we must allow `airflowctl dags list | jq ...<args>` style workflows for automation in custom scripting. Given we support json/yaml outputs, writing debug messages to stdout will break such usecases. Users maybe in a dev env when building such automations and could be using debug mode hence the need for this. > > Also we aren't making this an error, we are just printing the output to stderr instead of stdout. I understood that it prevents some actions. Where are those actions needed? In the case we don't use debug mode this won't be printed right? So other than integration tests, I see no further usage of insecurely saving token to a file :thinking: -- 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]
