You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by lu...@apache.org on 2005/04/26 00:06:30 UTC
cvs commit: jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/connector Response.java
luehe 2005/04/25 15:06:30
Modified: catalina/src/share/org/apache/catalina/connector
Response.java
Log:
Fix for Bugzilla 32604 ("Some httpHeaders can be lost in response").
Fix assumes that SRV.5.5, 2nd bullet, will be amended in the upcoming
Servlet Maintenance release, as follows:
The amount of content specified in the setContentLength method of the
response [ADD: has been greater than zero] and has been written to the
response
Revision Changes Path
1.12 +2 -2 jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/connector/Response.java
Index: Response.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/connector/Response.java,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -r1.11 -r1.12
--- Response.java 31 Mar 2005 10:31:53 -0000 1.11
+++ Response.java 25 Apr 2005 22:06:30 -0000 1.12
@@ -315,7 +315,7 @@
*/
public boolean isAppCommitted() {
return (this.appCommitted || isCommitted() || isSuspended()
- || ((getContentLength() != -1)
+ || ((getContentLength() > 0)
&& (getContentCount() >= getContentLength())));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org