tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cos...@apache.org
Subject cvs commit: jakarta-tomcat/src/facade22/org/apache/tomcat/facade RequestDispatcherImpl.java
Date Sat, 25 Aug 2001 18:11:12 GMT
costin      01/08/25 11:11:12

  Modified:    src/facade22/org/apache/tomcat/facade
                        RequestDispatcherImpl.java
  Log:
  Fix for 3162, requestDispatcher.forward must reset PathInfo and the other
  path related info in the passed request.
  
  Revision  Changes    Path
  1.20      +7 -1      jakarta-tomcat/src/facade22/org/apache/tomcat/facade/RequestDispatcherImpl.java
  
  Index: RequestDispatcherImpl.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/RequestDispatcherImpl.java,v
  retrieving revision 1.19
  retrieving revision 1.20
  diff -u -r1.19 -r1.20
  --- RequestDispatcherImpl.java	2001/08/21 04:56:44	1.19
  +++ RequestDispatcherImpl.java	2001/08/25 18:11:12	1.20
  @@ -222,12 +222,18 @@
   
   	// set the context - no need to fire context parsing again
   	realRequest.setContext( context );
  -
  +	
   	realRequest.requestURI().setString( context.getPath() + path );
  +	// # 3162
  +	realRequest.unparsedURI().recycle();
  +	//realRequest.query().recycle();
  +	realRequest.servletPath().recycle();
  +	realRequest.pathInfo().recycle();
   
   	// merge query string as specified in specs - before, it may affect
   	// the way the request is handled by special interceptors
   	if( queryString != null ) {
  +	    realRequest.queryString().setString(queryString);
   	    // Append queryString to the request parameters -
   	    // the original request is changed.
   	    realRequest.parameters().processParameters( queryString ); 
  
  
  

Mime
View raw message