potiuk commented on issue #17489:
URL: https://github.com/apache/airflow/issues/17489#issuecomment-894851965


   @jedcunningham @kaxil : from what I can see (and what seems to be confirmed 
by https://github.com/pallets/jinja/issues/17 - it seems that the "urlquery" 
filter we use is wrong.  It should be more complex. The problem is that 
urlquery replaces ' ' with '+' (and / with %2F) but 'urlencode` replaces " " 
with %20 but it does not replace "/" with %2F. I looked it up and seems that 
however strange it is, it is how it is. Unfortunately both user and password 
require STRICT percent-encoding.
   
   Seems that for Jinja, this is the only right way we can do it::
   
   ```
   {{ value|urlencode|replace("/", "%2f") }}
   ```
   


-- 
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-unsubscr...@airflow.apache.org

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


Reply via email to