You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Justin Edelson (JIRA)" <ji...@apache.org> on 2010/10/16 17:01:23 UTC
[jira] Commented: (SLING-1846) URL decoding
[ https://issues.apache.org/jira/browse/SLING-1846?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12921704#action_12921704 ]
Justin Edelson commented on SLING-1846:
---------------------------------------
It sounds like you're saying this is a bug in the Felix HttpService implementation. If so, it should be fixed there.
> URL decoding
> ------------
>
> Key: SLING-1846
> URL: https://issues.apache.org/jira/browse/SLING-1846
> Project: Sling
> Issue Type: Bug
> Reporter: Pierre-Luc Rigaux
>
> When we send an encoded url it is not decoded by Sling and it result by a 404 answer.
> Exemple
> "/test/My%20story.doc" should be decoded to "/test/My%20story.doc"
>
> "%E9%9F%B3%E4%B9%90.mp3" should be decoded to "音乐.mp3"
> My workaround was to add this code to org.apache.sling.engine.impl.SlingHttpServletRequestImpl.SlingHttpServletRequestImpl(RequestData, HttpServletRequest) construtor
> // prepare the pathInfo property
> String pathInfo = servletRequest.getServletPath();
> if (servletRequest.getPathInfo() != null) {
> pathInfo = pathInfo.concat(servletRequest.getPathInfo());
> try {
> pathInfo = URLDecoder.decode(pathInfo, "UTF-8");
> } catch (UnsupportedEncodingException e) {
> // TODO Auto-generated catch block
> e.printStackTrace();
> }
> }
> this.pathInfo = pathInfo;
> By checking a bit the method javax.servlet.http.HttpServletRequest.getServletPath() should return a decoded url, witch it is not the case. I find out that implementation was the felix one, but I didn't go further.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.