tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Remy Maucherat" <>
Subject Re: cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/modules/generators
Date Tue, 16 Apr 2002 09:19:46 GMT
> Hi Bill,
> I'm afraid your commit won't solve the problem. As stated in the original
> bug report, the problem occurs when the
> attribute is _present_. I've never looked at the 3.3 source, but it seems
> you only changed a case when the attribute is _not_ present.
> Let me quickly repeat the problem:
> /jsp/including.jsp --- includes ---> /jsp/test.jsp --- forwards to --->
> /html/test.html
> 1. When doing the forward, "javax.servlet.include.servlet_path" is set to
> "/jsp/test.jsp" (which is the correct behaviour).
> 2. Tomcat determines that "/html/test.html" is a static resource (also
> correct).
> 3. When serving the content, Tomcat determines the file to serve by first
> looking at "javax.servlet.include.servlet_path" and _only if this is not
> present_ calling getServletPath.
> The result is that Tomcat (at least 4.x) serves /jsp/test.jsp as a static
> resource (which it isn't). I still think the only way to reliably solve
> problem is to introduce an additional request attribute (please look at my
> bugreport for details). Otherwise, you can never be sure that deeply
> include/forward calls are handled correctly.
> I beliebe this is a serious issue and deserves a quick - but not dirty
;-) -
> fix. As said before, I'll happily supply a patch (with some help, I could
> also do it for 3.3). However, since my proposed solution would require a
> of files to be changed, I'd first like some opinions. Could the 4.x guys
> please look at it? I know it's a lot to read, but the problem and proposed
> fix are too complex to explain in just a few lines.

As I stated in the comments of the bug, I don't agree with your
interpretation about the JSP displaying "code".


To unsubscribe, e-mail:   <>
For additional commands, e-mail: <>

View raw message