Fixes #5888 - Limit usage of HTTP/2 connections.

jetty / jetty.project

Eclipse Jetty® - Web Container & Clients - supports HTTP/2, HTTP/1.1, HTTP/1.0, websocket, servlets, and more

https://eclipse.dev/jetty

Other

3.86k stars 1.91k forks source link

Closed sbordet closed 2 weeks ago

sbordet commented 4 weeks ago

Made the high-level HttpConnectionOverHTTP2 implement MaxUsable, so it cannot be used to open more streams than allowed.
Implemented low-level handling of explicit stream ids provided by applications.
Implemented low-level handling of stream id overflow.
Added test cases.

lorban commented 3 weeks ago

Overall, LGTM but these changes introduce a bug caught by PriorityTest.

sbordet commented 2 weeks ago

Note to self:

Using a placeholder stream is nice for idle timeouts, but messes with the local stream count.
Using a side map seems simpler, but requires timeout handling