Skip to content

Conversation

karthiknayak59
Copy link

@karthiknayak59 karthiknayak59 commented Aug 28, 2025

Motivation:

we found that MaxResetFramesTest.java fails under the following conditions:

Total number of requests is in the range (0, 6)
http2MaxResetFramesPerWindow is set to 2
Assert connections opened: 2
Assert connections closed: 2

My understanding is that the 3rd request should close the first connection, while the 4th request should create a new connection. This second connection should then be closed by the 6th request. Therefore, the test should conclude with 2 opened and 2 closed connections. Is my understanding correct? But we noticed that only one connection is opened and closed.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ikhoon
Copy link
Contributor

ikhoon commented Aug 31, 2025

My understanding is that the 3rd request should close the first connection, while the 4th request should create a new connection.

If the requests were executed sequentially, your assumption would be correct.
However, the test sends requests in parallel. It first sends 11 headers over the HTTP/2 connection, which in turn triggers RST_STREAM frames.
As a result, one connection should be closed in the test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants