Limess edited a comment on issue #17828:
URL: https://github.com/apache/airflow/issues/17828#issuecomment-906240516


   1. We set via environment variable: `AIRFLOW__CELERY__BROKER_URL: 
redis://redis-analytics-airflow.signal:6379/1`
   2. I get this answer: `'redis://redis-analytics-airflow.signal:6379/1'`
   
   From what I can tell the regex is wrong for host if there's no username or 
password, e.g
   
   https://regex101.com/r/kEohzC/1
   
   The host is in the second capture group in this instance, but the entrypoint 
is expecting it in the fourth. The port is being inferred entirely from the 
`redis://` scheme.
   
   I'd suggest this change:
   
   ```
   ([^:]*):\/\/(?:([^:@]*):?([^@]*)@)?([^\/:]*):?([0-9]*)\/([^\?]*)\??(.*)
   ```
   
   i.e. putting the user/pass part in it's own optional non-capture group - 
however that's not supported by bash regex as far as I'm aware


-- 
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]


Reply via email to