diff options
| author | Damien Neil <dneil@google.com> | 2026-03-13 14:43:22 -0700 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2026-03-16 10:38:58 -0700 |
| commit | 48bbe7996d1e53c39188fba941bc3f9e523c8b94 (patch) | |
| tree | e1650e1f7b2f5727d5efbde7d260fd614aa702f5 /src/cmd/compile/internal/noder | |
| parent | 6614616b7576a8011053c4b50fbb5e64d469837b (diff) | |
| download | go-48bbe7996d1e53c39188fba941bc3f9e523c8b94.tar.xz | |
net/http/internal/http2: deflake TestServer_Rejects_Too_Many_Streams
This test contains a race condition in the server handler:
inHandler <- streamID
<-leaveHandler
We assume that all requests queue reading from leaveHandler in order,
but it is possible for the second request (stream id 3) to arrive at
leaveHandler before the first (stream id 1).
We could fix the race with a judicious synctest.Wait, but rewrite
the test to use serverHandlerCall to manipulate server handlers,
which permits us to precisely pick which request to unblock.
Fixes #78117
Change-Id: Icd393c81e019a0b5176976a6b50145a26a6a6964
Reviewed-on: https://go-review.googlesource.com/c/go/+/755320
Reviewed-by: Nicholas Husin <nsh@golang.org>
Reviewed-by: Nicholas Husin <husin@google.com>
Auto-Submit: Nicholas Husin <husin@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/cmd/compile/internal/noder')
0 files changed, 0 insertions, 0 deletions