[GitHub] [airflow] potiuk commented on issue #20063: Forward slash in `dag_run_id` gives rise to trouble accessing things through the REST API

2021-12-12 Thread GitBox
potiuk commented on issue #20063: URL: https://github.com/apache/airflow/issues/20063#issuecomment-991967998 Agree. Sounds like a hack. I also thought about it and it's actually kinda misleading to use `/` for different purposes in a URL. Also if we use %2F, its none better. Good e

[GitHub] [airflow] potiuk commented on issue #20063: Forward slash in `dag_run_id` gives rise to trouble accessing things through the REST API

2021-12-09 Thread GitBox
potiuk commented on issue #20063: URL: https://github.com/apache/airflow/issues/20063#issuecomment-990198956 Thanks @uranusjr @andrewgodwin. TIL. I believe (WDYT) the best way to "fix" it is to warn peeople (and deprecate) the use of "/" in the ID and explain that they won't be able

[GitHub] [airflow] potiuk commented on issue #20063: Forward slash in `dag_run_id` gives rise to trouble accessing things through the REST API

2021-12-09 Thread GitBox
potiuk commented on issue #20063: URL: https://github.com/apache/airflow/issues/20063#issuecomment-989657584 Bummer -- 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 uns

[GitHub] [airflow] potiuk commented on issue #20063: Forward slash in `dag_run_id` gives rise to trouble accessing things through the REST API

2021-12-08 Thread GitBox
potiuk commented on issue #20063: URL: https://github.com/apache/airflow/issues/20063#issuecomment-989206485 Yep. It definitely should -- 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 specifi

[GitHub] [airflow] potiuk commented on issue #20063: Forward slash in `dag_run_id` gives rise to trouble accessing things through the REST API

2021-12-06 Thread GitBox
potiuk commented on issue #20063: URL: https://github.com/apache/airflow/issues/20063#issuecomment-986738954 Agreee. The %2F is the only "good" way to go. And if it does not work - I think we should probably fix it. This might become handdy when we add multi-tenancy. "/" seems to be for ex