[GitHub] wdhorton edited a comment on issue #3721: [AIRFLOW-2875] Escape env vars in tmp config

[GitBox <gi...@apache.org>]

wdhorton edited a comment on issue #3721: [AIRFLOW-2875] Escape env vars in tmp config
URL: https://github.com/apache/incubator-airflow/pull/3721#issuecomment-411796810
 
 
   This doesn't affect values that are in the config file, just ones that are being pulled from the env (like `AIRFLOW__CELERY__BROKER_URL` in my case) and written to the tmp config file with `tmp_configuration_copy()`. So `%` interpolation in the config file should continue to work as is.
   
   That said, it looks like I broke a bunch of tests, so I need to figure out how this is having an effect in other places that I didn't expect.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services