Skip to content

Close the HTTP/2 test server listen socket after accepting - #541

Open
ericmj wants to merge 1 commit into
fuzz-http1-mid-scenario-requestsfrom
fuzz-http2-close-listen-socket
Open

ericmj wants to merge 1 commit into
fuzz-http1-mid-scenario-requestsfrom
fuzz-http2-close-listen-socket

Conversation

@ericmj

@ericmj ericmj commented Oct 16, 2026

Copy link
Copy Markdown
Member

TestServer.listen_and_accept/0 opened a TLS listen socket that nothing closed, so every HTTP/2 fuzz scenario left a listener behind: 200 scenarios grew the tcp_inet ports from 1 to 201 and the processes from 125 to 527. The accepting task now closes the listen socket once it has accepted the single connection it serves, and the same 200 scenarios end with 1 port and 127 processes.

TestServer.listen_and_accept/0 opened a TLS listen socket that nothing
closed, so every HTTP/2 fuzz scenario left a listener behind: 200
scenarios grew the tcp_inet ports from 1 to 201 and the processes from
125 to 527. The accepting task now closes the listen socket once it has
accepted the single connection it serves, and the same 200 scenarios end
with 1 port and 127 processes.
@ericmj
ericmj added this pull request to stack #544 October 16, 2026 13:10
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 89

Coverage increased (+0.3%) to 90.711%

Details

  • Coverage increased (+0.3%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1970
Covered Lines: 1787
Line Coverage: 90.71%
Coverage Strength: 710.7 hits per line

馃挍 - Coveralls

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.

2 participants