Decoded URI in equinox [message #1759639] |
Fri, 14 April 2017 10:39 |
Arnaud MERGEY Messages: 243 Registered: March 2010 Location: France |
Senior Member |
|
|
Upgrading from mars to neon, we were facing strange issue in a rest api with params with special characters
After investigating it, we discover we are exactly facing this bug https://bugs.eclipse.org/bugs/show_bug.cgi?id=497510
Checking in master there is a comment in code :
// BUGBUG this should probably be URI encoded?
// Servlet 2.2 Section 5.4
According to servlet API I confirm it should not be decoded there, and it is a regression introduced in Neon.
It seems quite easy to fix
in HttpServletRequestWrapperImpl::parseRequestLine
reqURI should not be decoded.
I could patch it, but I would like to know, were there any reasons to decode the uri ? and do you know why it was done like this ?
|
|
|
Powered by
FUDForum. Page generated in 0.03373 seconds