net/http: `TestTransportPersistConnReadLoopEOF` failures with `Connection reset by peer` on wasip1

[bcmills]

#!watchflakes
post <- pkg == "net/http" && test == "TestTransportPersistConnReadLoopEOF" && `send: [Cc]onnection reset by peer`

Examples:

• https://build.golang.org/log/db36c47e207d169418b1fb6cf8e729fc179ebeb9
• https://build.golang.org/log/a384a8b3d229e534d42898bb547f5b7855d59205

--- FAIL: TestTransportPersistConnReadLoopEOF (0.00s)
    transport_internal_test.go:62: pc.closed = http.nothingWrittenError{error:(*net.OpError)(0x181c5a0)}, write tcp 127.0.0.1:2->127.0.0.1:1: send: Connection reset by peer; want errServerClosedIdle or transportReadFromServerError
FAIL
FAIL	net/http	28.029s

See previously:

• net/http: "use of closed network connection" in TestTransportPersistConnReadLoopEOF on android-386-emu #49450
• net/http: TestServerEmptyBodyRace_h2 flakes with "connection reset by peer" #22540

(CC @neild @golang/wasm)

[bcmills]

Ah, I think this is related to #30938.

https://go.dev/cl/379554 changed the propagation of nothingWrittenError and also changed an earlier assertion in TestTransportPersistConnReadLoopEOF to allow it.

[gopherbot]

Sorry, but there were parse errors in the watch flakes script.
The script I found was:

#!watchflakes
post <- pkg == "net/http" && test == "TestTransportPersistConnReadLoopEOF" && `send: [Cc]onnection reset by peer'

And the problems were:

script:2.79: newline in backquoted regexp

See https://go.dev/wiki/Watchflakes for details.

— watchflakes

[bcmills]

Hmm. I wonder if we just need to crank up rstAvoidanceDelay for this test. (Search for SetRSTAvoidanceDelay and apply a similar loop, perhaps?)

[gopherbot]

Change https://go.dev/cl/558595 mentions this issue: net/http: ignore nothingWrittenError in TestTransportPersistConnReadLoopEOF

[gopherbot]

Change https://go.dev/cl/558915 mentions this issue: net: in the fake implementation, allow writes to buffer on closed connections