You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2016/06/30 11:50:08 UTC
svn commit: r1750776 - /tomcat/trunk/java/org/apache/coyote/http2/Stream.java
Author: markt
Date: Thu Jun 30 11:50:07 2016
New Revision: 1750776
URL: http://svn.apache.org/viewvc?rev=1750776&view=rev
Log:
Align HTTP/2 with AJP and HTTP/1.1 and commit the response before any body data is written to OutputBuffer
Modified:
tomcat/trunk/java/org/apache/coyote/http2/Stream.java
Modified: tomcat/trunk/java/org/apache/coyote/http2/Stream.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http2/Stream.java?rev=1750776&r1=1750775&r2=1750776&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http2/Stream.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http2/Stream.java Thu Jun 30 11:50:07 2016
@@ -467,6 +467,9 @@ public class Stream extends AbstractStre
throw new IllegalStateException(
sm.getString("stream.closed", getConnectionId(), getIdentifier()));
}
+ if (!coyoteResponse.isCommitted()) {
+ coyoteResponse.sendHeaders();
+ }
int len = chunk.getLength();
int offset = 0;
while (len > 0) {
@@ -497,9 +500,6 @@ public class Stream extends AbstractStre
getIdentifier(), Integer.toString(buffer.position()),
Boolean.toString(writeInProgress), Boolean.toString(closed)));
}
- if (!coyoteResponse.isCommitted()) {
- coyoteResponse.sendHeaders();
- }
if (buffer.position() == 0) {
if (closed && !endOfStreamSent) {
// Handling this special case here is simpler than trying
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org