billbarker    02/03/18 22:06:18

  Modified:    coyote/src/java/org/apache/coyote/tomcat3
                        CoyoteInterceptor.java
  Log:
  Don't stop the processing if the response is already committed.
  
  Letting the IllegalStateException through was preventing error handling from working 
if e.g. the servlet throws an exception after the response is committed.  This in turn 
prevents the useful stack trace from showing up on development servers.  Under normal 
use, the response should never be committed at this point on the primary request.
  
  Revision  Changes    Path
  1.5       +2 -2      
jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3/CoyoteInterceptor.java
  
  Index: CoyoteInterceptor.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat3/CoyoteInterceptor.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- CoyoteInterceptor.java    19 Mar 2002 04:27:50 -0000      1.4
  +++ CoyoteInterceptor.java    19 Mar 2002 06:06:18 -0000      1.5
  @@ -276,8 +276,8 @@
        if(response instanceof CoyoteResponse) {
            try {
                ((CoyoteResponse)response).sendAcknowledgement();
  -         } catch(IOException iex) {
  -             log("Can't send ACK",iex);
  +         } catch(Exception ex) {
  +             log("Can't send ACK", ex);
            }
        }
        return 0;
  
  
  

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

Reply via email to