mikyll commented on PR #11788:
URL: https://github.com/apache/apisix/pull/11788#issuecomment-2517766817
@shreemaan-abhishek thank you for the reply, but I think you misunderstand
my previous comment... I **agree** that `/` should only be treated as a **path
separator**, it wouldn't make sense otherwise and it would also be a security
issue.
**_However_**, that's exactly the reason why we need URL-encoding: when we
URL-encode
("[percent-encoding](https://developer.mozilla.org/en-US/docs/Glossary/Percent-encoding)")
characters, they are treated as **raw data**.
Currently, APISIX with `radixtree_uri_with_parameter` **doesn't fully
support this**, since the router fails to route requests that contain `%2F` in
a path parameter.
This PR fixed the problem for many special characters but **not** for `%2F`
(encoded `/`)!
See the [comparison
table](https://github.com/apache/apisix/pull/11788#issuecomment-2505712426)
above.
Please see [issue #11810](https://github.com/apache/apisix/issues/11810) for
further information.
--
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]
