cvs commit: jakarta-tomcat/src/facade22/org/apache/tomcat/facade HttpServletResponseFacade.java

[billbarker]

billbarker    02/01/29 19:37:12

  Modified:    src/facade22/org/apache/tomcat/facade
                        HttpServletResponseFacade.java
  Log:
  Don't encode URLs that only have an anchor.
  
  Now response.encodeURL("#foobar") works as well.
  
  Fix for bug 6115
  Reported by: Peter Nuetzel [EMAIL PROTECTED]
  
  Revision  Changes    Path
  1.24      +7 -5      
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletResponseFacade.java
  
  Index: HttpServletResponseFacade.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletResponseFacade.java,v
  retrieving revision 1.23
  retrieving revision 1.24
  diff -u -r1.23 -r1.24
  --- HttpServletResponseFacade.java    16 Nov 2001 04:10:35 -0000      1.23
  +++ HttpServletResponseFacade.java    30 Jan 2002 03:37:12 -0000      1.24
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletResponseFacade.java,v
 1.23 2001/11/16 04:10:35 billbarker Exp $
  - * $Revision: 1.23 $
  - * $Date: 2001/11/16 04:10:35 $
  + * $Header: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpServletResponseFacade.java,v
 1.24 2002/01/30 03:37:12 billbarker Exp $
  + * $Revision: 1.24 $
  + * $Date: 2002/01/30 03:37:12 $
    *
    * ====================================================================
    *
  @@ -447,8 +447,10 @@
            path = path.substring(0,hashP);
        }
        StringBuffer sb = new StringBuffer(path);
  -     sb.append(";jsessionid=");
  -     sb.append(sessionId);
  +     if( sb.length() > 0) { // Can't have jsessionid first.
  +         sb.append(";jsessionid=");
  +         sb.append(sessionId);
  +     }
        if(anchor != null) 
            sb.append(anchor);
        if (query != null)
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>