Skip to content

fix(conn): cap connections to minimum needed#874

Open
tuliperis wants to merge 1 commit intohatoo:masterfrom
tuliperis:conn
Open

fix(conn): cap connections to minimum needed#874
tuliperis wants to merge 1 commit intohatoo:masterfrom
tuliperis:conn

Conversation

@tuliperis
Copy link

@tuliperis tuliperis commented Mar 9, 2026

This PR fixes #864

Instead of c = min(n, c), we take into account p: c = min(ceil(n / p), c)

Manually tested:

cargo run --features http3 -- --insecure --http-version {1.1,2,3} https://localhost/ -n 3 -q 3 -p 2

2 per protocol

image

3 per protocol

cargo run --features http3 -- --insecure --http-version {1.1,2,3} https://localhost/ -n 3 -q 3
image

1 connection

cargo run --features http3 -- --insecure --http-version 2 https://localhost/ -n 3 -c 50 -p 3
image

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.

oha opens more connections than needed

1 participant