tree 3425442ab9a97b92d439ba14ee60e88d4a6b28ab
parent e13a2654a71e8d479c387f1c45c295c956910cf7
author Damien Neil <dneil@google.com> 1634146171 -0700
committer Damien Neil <dneil@google.com> 1634232344 +0000

http2: deflake TestTransportReqBodyAfterResponse_200

Don't send the END_STREAM flag from the server until the client
sends its END_STREAM. Avoids a flaky failure when the client
sends the END_STREAM in a zero-length DATA frame:

  - The server reads bodySize bytes and half-closes the stream.
  - The client's Response.Body returns EOF.
  - The test calls Response.Body.Close.
  - The client resets the stream.

Avoid hanging forever on the client side of the test if the
server side exits with an error.

Fixes golang/go#48970

Change-Id: Ic921a3f60149abbb5434ec7a53985becea7b23c3
Reviewed-on: https://go-review.googlesource.com/c/net/+/355649
Trust: Damien Neil <dneil@google.com>
Reviewed-by: Bryan C. Mills <bcmills@google.com>