You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Felix Meschberger (JIRA)" <ji...@apache.org> on 2014/02/17 17:22:23 UTC

[jira] [Resolved] (SLING-3393) Parameter re-encoding does not work for Jetty 7 and above

     [ https://issues.apache.org/jira/browse/SLING-3393?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Felix Meschberger resolved SLING-3393.
--------------------------------------

       Resolution: Fixed
    Fix Version/s: Engine 2.2.12

Thanks for providing the patch. I have applied it (slightly reformatted) in Rev. 1569040.

Please note, that the approach discussed on the Sling dev list [Request Parameter Themes|http://sling.markmail.org/thread/iw5lxsb6ewpefvvv] might render this patch obsolete. At least it solves the short term problem.

> Parameter re-encoding does not work for Jetty 7 and above
> ---------------------------------------------------------
>
>                 Key: SLING-3393
>                 URL: https://issues.apache.org/jira/browse/SLING-3393
>             Project: Sling
>          Issue Type: Bug
>          Components: Engine
>    Affects Versions: Engine 2.2.10
>            Reporter: Dominique Pfister
>            Assignee: Felix Meschberger
>             Fix For: Engine 2.2.12
>
>         Attachments: SLING-3393.patch
>
>
> SLING-559 changed the query encoding to 8859-1 when used in combination with Jetty, by setting a request attribute called:
>     "org.mortbay.jetty.Request.queryEncoding"
> With Jetty 7 and above, this attribute has been changed to: 
>     "org.eclipse.jetty.server.Request.queryEncoding" 
> [1], so the fix no longer works. I suggest to either use both attribute names or only the new one, if support for Jetty 6 or earlier is no longer a must.
> [1] http://download.eclipse.org/jetty/stable-7/apidocs/org/eclipse/jetty/server/Request.html#setQueryEncoding(java.lang.String)



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)