You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2003/08/04 22:14:36 UTC
cvs commit: jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet JspServlet.java
remm 2003/08/04 13:14:36
Modified: jasper2/src/share/org/apache/jasper/servlet JspServlet.java
Log:
- Fix including a JSP, where Jasper would do weird things whenever the
outer request had a non null pathInfo. now, it will use the included servlet
path correctly.
- This fixes bug 21206 (issue with Jetspeed).
Revision Changes Path
1.29 +10 -11 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java
Index: JspServlet.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java,v
retrieving revision 1.28
retrieving revision 1.29
diff -u -r1.28 -r1.29
--- JspServlet.java 22 Jul 2003 21:04:46 -0000 1.28
+++ JspServlet.java 4 Aug 2003 20:14:36 -0000 1.29
@@ -202,18 +202,17 @@
if (includeUri == null) {
jspUri = request.getServletPath();
+ // When jsp-property-group/url-matching is used, and when the
+ // jsp is not defined with <servlet-name>, the url
+ // as to be passed as it is to the JSP container (since
+ // Catalina doesn't know anything about the requested JSP
+ if (request.getPathInfo() != null) {
+ jspUri = request.getServletPath() + request.getPathInfo();
+ }
} else {
jspUri = includeUri;
}
- // When jsp-property-group/url-matching is used, and when the
- // jsp is not defined with <servlet-name>, the url
- // as to be passed as it is to the JSP container (since Catalina
- // doesn't know anything about the requested JSP
- if(request.getPathInfo() != null){
- jspUri = request.getServletPath() + request.getPathInfo();
- }
-
String jspFile = (String) request.getAttribute(Constants.JSP_FILE);
if (jspFile != null) {
jspUri = jspFile;