UrlRewriteFilter is not able to handle URLs with an encoded '%' about urlrewritefilter

[GoogleCodeExporter]

Could you provide a sample rule and request URL that demonstrates this please?

Original comment by [email protected] on 19 Aug 2008 at 8:50

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

To reproduce this problem I used this rule:

    <rule>
    <name>Proxy</name>
        <from>^(/bom-auth/.*)$</from>
        <to type="forward">/proxy$1</to>
    </rule>

and made a request to:

      /bom-auth/D%25%2cD.pdf

Which throws this exception:

===================================================================

javax.servlet.ServletException: unable to get request dispatcher for 
/proxy/bom-auth/D%,D.pdf
    at 
org.tuckey.web.filters.urlrewrite.NormalRewrittenUrl.getRequestDispatcher(Normal
RewrittenUrl.java:263)
    at org.tuckey.web.filters.urlrewrite.NormalRewrittenUrl.doRewrite(NormalRewrittenUrl.java:194)
    at org.tuckey.web.filters.urlrewrite.RuleChain.handleRewrite(RuleChain.java:159)
    at org.tuckey.web.filters.urlrewrite.RuleChain.doRules(RuleChain.java:141)
    at org.tuckey.web.filters.urlrewrite.UrlRewriter.processRequest(UrlRewriter.java:90)
    at org.tuckey.web.filters.urlrewrite.UrlRewriteFilter.doFilter(UrlRewriteFilter.java:406)
    at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1065)
        ...

===================================================================

Note the lack of encoding in the url as reported in the stack trace.

Original comment by [email protected] on 19 Aug 2008 at 10:58

[GoogleCodeExporter]

I'm altering decoding of the request.  The following test case will work once 
changes are checked in:

<urlrewrite decode-using="null">
    <rule>
        <name>Proxy</name>
        <from>^/no-decode-test/(.*)$</from>
        <to type="forward">/no-decode-test-$1.jsp</to>
    </rule>
</urlrewrite>

Request made for: /no-decode-test/D%25%2cD

Request is successfully forwarded to: no-decode-test-D%25%2cD.jsp

Original comment by [email protected] on 2 Dec 2008 at 9:55

• Changed state: Started

[GoogleCodeExporter]

Set milestone to 3.2.

Original comment by [email protected] on 2 Dec 2008 at 10:23

• Added labels: Milestone-Release3.2

[GoogleCodeExporter]

Done.   This will be in 3.2.

Original comment by [email protected] on 5 Dec 2008 at 2:41

• Changed state: Fixed