Skip to content

stream: enable autoDestroy by default #30623

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ronag wants to merge 8 commits into from
Closed

stream: enable autoDestroy by default #30623

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
df2d395
stream: enable autoDestroy by default
ronag Nov 24, 2019
3e188d7
fixup: linting
ronag Nov 24, 2019
d9ff59b
fixup: tests
ronag Nov 24, 2019
28cf38d
fixup: test
ronag Nov 24, 2019
1899ada
fixup: docs
ronag Nov 24, 2019
616c56e
fixup: tests + Readable capture
ronag Dec 25, 2019
ccdfafe
Apply suggestions from code review
ronag Jan 1, 2020
1bb3889
fixup
ronag Jan 1, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup: test
  • Loading branch information
@ronag
ronag committed Jan 1, 2020
commit 28cf38d36dd618c68994223852bd3e271d42583c
8 changes: 5 additions & 3 deletions lib/_http_incoming.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,15 @@ function readStop(socket) {

/* Abstract base class for ServerRequest and ClientResponse. */
function IncomingMessage(socket) {
const streamOptions = { autoDestroy: false };
let streamOptions;

if (socket) {
streamOptions.highWaterMark = socket.readableHighWaterMark;
streamOptions = {
highWaterMark: socket.readableHighWaterMark
};
}

Stream.Readable.call(this, streamOptions);
Stream.Readable.call(this, { autoDestroy: false, ...streamOptions });

this._readableState.readingMore = true;

Expand Down