billbarker    02/03/27 21:03:03

  Modified:    coyote/src/java/org/apache/coyote/tomcat3
                        CoyoteResponse.java
  Log:
  Fix problem with error-handling.
  
  If you attempt a PUT to a protected page, then the old code was sending an ACK, 
followed by a 401 (which confuses browsers).  Now it just sends the 401 and ACKs after 
you have logged in.
  
  Revision  Changes    Path
  1.7       +3 -1      
jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3/CoyoteResponse.java
  
  Index: CoyoteResponse.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3/CoyoteResponse.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- CoyoteResponse.java       19 Mar 2002 06:20:55 -0000      1.6
  +++ CoyoteResponse.java       28 Mar 2002 05:03:03 -0000      1.7
  @@ -143,6 +143,8 @@
       public void sendAcknowledgement()
           throws IOException {
   
  +     if( status >= 300 ) // Don't ACK on errors.
  +         acknowledged = true;
        // Don't ACK twice on the same request. (e.g. on a forward)
        if(acknowledged)
            return;
  @@ -153,7 +155,7 @@
               throw new IllegalStateException
                   (sm.getString("hsrf.error.ise"));
   
  -        coyoteResponse.acknowledge();
  +     coyoteResponse.acknowledge();
        acknowledged=true;
       }
   }
  
  
  

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

Reply via email to