search

private-octopus / picoquic

Minimal implementation of the QUIC protocol
MIT License
540 stars 159 forks source link

Stress parameters in picohttp_t #1496

Closed huitema closed 1 year ago

huitema commented 1 year ago

Rework the HTTP test picohttp_t so the parameters are easier to understand. The http stress test was creating a server context with a fixed number of connections. Fixed that, so the number of expected connection matches the test.

huitema commented 1 year ago

There were several observed cases of the "http_corrupt" test failing, even when the number of clients was set to a low number. This was most probably caused by the arrival of packets with spoofed connection IDs, creating extra connection contexts that would then saturate the server. This is an issue with QUIC, not HTTP3, so the natural solution was to fix the test to not randomly corrupt the first bytes of long packet headers.