net/http: TestRequestLimit/h2 becomes significantly more expensive and slower after x/net@v0.23.0 [1.22 backport]

[gopherbot]

@neild requested issue #66668 to be considered for backport to the next 1.22 minor release.

@gopherbot please open backport issues. This is a bug/deficiency in a previous cherrypick, so we should backport the fix as well.

[dmitshur]

should backport the fix as well

To clarify, do you think the fix to be backported should include changes in CL 576756 and CL 576895?

[neild]

Just https://go.dev/cl/576756 (sending the correct LastStreamID), which is a bugfix/refinement of the previously backported security fix. https://go.dev/cl/576895 is an improvement to the client behavior, I think, but not directly related.

[cagedmantis]

Approved as this is a serious performance regression that was introduced by a fix.

[gopherbot]

Change https://go.dev/cl/578338 mentions this issue: [internal-branch.go1.22-vendor] http2: send correct LastStreamID in stream-caused GOAWAY

[gopherbot]

Change https://go.dev/cl/578337 mentions this issue: [internal-branch.go1.22-vendor] http2: fix TestServerContinuationFlood flakes

[gopherbot]

Change https://go.dev/cl/578358 mentions this issue: [release-branch.go1.22] net/http: update bundled golang.org/x/net/http2

[gopherbot]

Closed by merging d6c972a to release-branch.go1.22.