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

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

andrewgodwin commented on issue #20063:
URL: https://github.com/apache/airflow/issues/20063#issuecomment-990042222


   Ah yes, this old biscuit. ASGI behaves the same in this regard - as HTTP specifies that `%2f` is equal to a `/`, ASGI also force-urldecodes everything before the application has a chance to look at the query string. `raw_path` is there for the people who complained about this, but in reality, you shouldn't expect to discriminate between escapes and the characters they represent (e.g. a proxy in between you and the client may muck with them).
    
   I'd suggest either double-escaping these values, which may prove a little tricky in terms of backwards compatibility, or fix the routing so it's an unambiguous path that can be routed by Flask.
   
   


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