David Halls 538d8ce106 stream: don't push null from closed promise #42694 ...

closed promise is subscribed to first so will be
resolved first, before any read promise.

This causes data after EOF error to be thrown.

Remove the push null from the closed promise handler.
The push null gets done from the read handler
when it detects done.

PR-URL: https://github.com/nodejs/node/pull/45026
Fixes: https://github.com/nodejs/node/issues/42694
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

2022-10-22 17:27:37 +00:00

..

adapters.js

stream: don't push null from closed promise #42694

2022-10-22 17:27:37 +00:00

compression.js

lib: move kEnumerableProperty to internal/util

2022-02-11 17:30:47 +00:00

encoding.js

stream: update TextEncoderStream to align the latest spec

2022-08-04 22:41:28 +00:00

queuingstrategies.js

stream: fix size function returned from QueuingStrategies

2022-10-12 10:26:35 +00:00

readablestream.js

lib: add lint rule to protect against Object.prototype.then pollution

2022-10-21 03:12:03 +00:00

transfer.js

lib: use null-prototype objects for property descriptors

2022-06-03 09:23:58 +01:00

transformstream.js

lib: use safe Promise alternatives when available

2022-08-27 11:37:01 +02:00

util.js

stream: add ReadableByteStream.tee()

2022-09-08 22:06:06 +00:00

writablestream.js

stream: remove abortReason from WritableStreamDefaultController

2022-09-09 07:10:12 +00:00