[jira] Commented: (TAPESTRY-2187) Context Parameter Encoding confuses Jetty Server, use non UrlEncoding.

["Howard M. Lewis Ship (JIRA)" <de...@tapestry.apache.org>]

    [ https://issues.apache.org/jira/browse/TAPESTRY-2187?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12575443#action_12575443 ] 

Howard M. Lewis Ship commented on TAPESTRY-2187:
------------------------------------------------

What do you mean by "confused Jetty"?  Our tests use this and it works correctly.

This is a sample complex URL:

http://localhost:8080/target/betty/wilma/betty%252Fwilma/%E8%8A%B1%E5%AD%90

This works properly with Jetty because Jetty is the server used with our unit tests.

> Context Parameter Encoding confuses Jetty Server, use non UrlEncoding.
> ----------------------------------------------------------------------
>
>                 Key: TAPESTRY-2187
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2187
>             Project: Tapestry
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.0.11
>            Reporter: Fernando
>
> TapestryInternalUtils.escapePercentAndSlash escapes the Percent and Slash characters using URL encoding.  This confuses Jetty, and could confuse browsers and other servers because of unknown times of encoding/decoding done by browsers and servers...
> I propose to change the constants in TapestryInternalUtils:
>     private static final String ENCODED_PERCENT = "%25";
>     private static final String ENCODED_SLASH = "%2F";
> to use a different escape character, maybe one of the unreserved characters: "-", "_", ".", "~"; then make sure to encode that character as well..
> so ENCODED_PERCENT would be ~25, ENCODED_SLASH would be ~2F
> and we would add ENCODED_TILDE to be ~7E
> http://en.wikipedia.org/wiki/Percent-encoding

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org