tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: svn commit: r1005192 - /tomcat/tc6.0.x/trunk/STATUS.txt
Date Wed, 13 Oct 2010 15:57:23 GMT
2010/10/6  <markt@apache.org>:
> Author: markt
> Date: Wed Oct  6 18:03:03 2010
> New Revision: 1005192
>
> URL: http://svn.apache.org/viewvc?rev=1005192&view=rev
> Log:
> Proposal
>
> Modified:
>    tomcat/tc6.0.x/trunk/STATUS.txt
>
> +
> +* Fix path parameter handling. Currently the following URL fails with a 404:
> +  http://localhost:8080/examples/jsp/snp;x=y/snoop.jsp
> +  http://people.apache.org/~markt/patches/2010-10-06-path-param-tc6.patch
> +  +1: markt
> +  -1:
>

I think this is wrong, because a path parameter is not just a pair of
(name, value), but a triple of (path, name, value),  taking into
account the path segment where it was applied.

The proposed patch strips information on the parameters from the path.
How the application is supposed to have access to them?

Best regards,
Konstantin Kolinko

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


Mime
View raw message