This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new bfd87b1214 Tighter validation to debug Gump failure
bfd87b1214 is described below

commit bfd87b1214de50c515358692f6e5afac5ae9e383
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Sun Aug 6 13:14:14 2023 +0100

    Tighter validation to debug Gump failure
---
 test/org/apache/coyote/http2/TestHttp2Section_6_9.java | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/test/org/apache/coyote/http2/TestHttp2Section_6_9.java 
b/test/org/apache/coyote/http2/TestHttp2Section_6_9.java
index 176c669cef..966bf88d24 100644
--- a/test/org/apache/coyote/http2/TestHttp2Section_6_9.java
+++ b/test/org/apache/coyote/http2/TestHttp2Section_6_9.java
@@ -93,12 +93,20 @@ public class TestHttp2Section_6_9 extends Http2TestBase {
         // Default connection window size is 64k - 1. Initial request will have
         // used 8k (56k -1).
 
-        // Use up the remaining connection window. These requests require 56k
-        // but there is only 56k - 1 available.
-        for (int i = 3; i < 17; i += 2) {
+        // Use up the remaining connection window. These requests require 48k.
+        for (int i = 3; i < 15; i += 2) {
             sendSimpleGetRequest(i);
             readSimpleGetResponse();
+            Assert.assertEquals(getSimpleResponseTrace(i), output.getTrace());
+            output.clearTrace();
         }
+        // This request requires 8k but there is only 8k-1 available
+        sendSimpleGetRequest(15);
+        readSimpleGetResponse();
+        String expected = getResponseBodyFrameTrace(15, 200, 
"application/octet-stream", null, "8191", "8192");
+        // No end of stream
+        expected = expected.substring(0, expected.length() - 
"15-EndOfStream\n".length());
+        Assert.assertEquals(expected, output.getTrace());
         output.clearTrace();
 
         // It should be possible to send a request that generates an empty


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

Reply via email to