Author: markt
Date: Wed Aug  8 09:40:37 2018
New Revision: 1837637

URL: http://svn.apache.org/viewvc?rev=1837637&view=rev
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=62605
Ensure ReadListener.onDataAvailable() is called when the initial request body 
data arrives after the request headers when using asynchronous processing over 
HTTP/2.

Modified:
    tomcat/trunk/java/org/apache/coyote/http2/Stream.java
    tomcat/trunk/webapps/docs/changelog.xml

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=1837637&r1=1837636&r2=1837637&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http2/Stream.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http2/Stream.java Wed Aug  8 09:40:37 
2018
@@ -908,10 +908,10 @@ class Stream extends AbstractStream impl
 
 
         final void registerReadInterest() {
-            if (inBuffer != null) {
-                synchronized (inBuffer) {
-                    readInterest = true;
-                }
+            ensureBuffersExist();
+
+            synchronized (inBuffer) {
+                readInterest = true;
             }
         }
 

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1837637&r1=1837636&r2=1837637&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Wed Aug  8 09:40:37 2018
@@ -138,6 +138,11 @@
         <bug>62526</bug>: Correctly handle PKCS12 format key stores when the 
key
         store password is configured to be the empty string. (markt)
       </fix>
+      <fix>
+        <bug>62605</bug>: Ensure <code>ReadListener.onDataAvailable()</code> is
+        called when the initial request body data arrives after the request
+        headers when using asynchronous processing over HTTP/2. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Jasper">



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to