tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andreas Junghans" <>
Subject Re: cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/modules/generators
Date Tue, 16 Apr 2002 08:35:32 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 "javax.servlet.include.servlet_path"
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 --->

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
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 this
problem is to introduce an additional request attribute (please look at my
bugreport for details). Otherwise, you can never be sure that deeply nested
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 lot
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.

Best regards

  Andreas Junghans

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

View raw message