[GitHub] [airflow] potiuk commented on pull request #33902: allow optional password replacement

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on PR #33902:
URL: https://github.com/apache/airflow/pull/33902#issuecomment-1704358422

   This is quite certainly wrong approach. 
   
   You do not need to do it in such complex way. You should use `_CMD` version of the variables for that.
   
   You can have `BROKER_URL_PASSWORD` env variable password and have a script in your environment to build the broker url out of that:
   
   ```
   #!/bin/bash
   echo -n "amqp://user:${BROKER_URL_PASSWORD}@rabbitmq:5672/
   ```
   
   save it in `/usr/local/bin/get_broker_url`
   
   And set `AIRFLOW__CELERY__BROKER_URL_CMD= 
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org