sjyangkevin commented on PR #55704: URL: https://github.com/apache/airflow/pull/55704#issuecomment-3300844519
Thanks @pierrejeambrun , @jason810496 > I think we should always unquote. > can you add a test where the target url is “relative” to the base url (I.e nested down the path). > I think with the current code that wouldn’t work, but it should. > For instance: "https%3A%2F%2Frequesting_server_base_url.com%2Fprefix2%2Fsub_path” Yes. In the case of target url is "relative" to the base url, the additional check there will fail to unquote the target url. Thanks for pointing this out. -- 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]
