FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

8.2.1 proposal by Fishrock123 · Pull Request #14399 · nodejs/node · GitHub

/ node Public

8.2.1 proposal - #14399

Merged
Fishrock123 merged 4 commits into
v8.xfrom
v8.2.1-proposal
Jul 20, 2017
Merged

8.2.1 proposal#14399
Fishrock123 merged 4 commits into
v8.xfrom
v8.2.1-proposal

Conversation

Copy link
Copy Markdown
Contributor

"Emergency" patch release. Today.

Primarily for #14387, but also including #14396 which didn't make it into 8.2.0.

If the stream is destroyed while the transform is still being
applied, push() should not be called, and the internal state
should be cleared.

Refs: koajs/compress#60
PR-URL: #14330
Backport-PR-URL: #14396
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

 Conflicts:
	lib/zlib.js
Fishrock123 added meta Issues and PRs related to the general management of the project. v8.x labels Jul 20, 2017
Fishrock123 self-assigned this Jul 20, 2017
nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. labels Jul 20, 2017
Fishrock123 changed the base branch from master to v8.x July 20, 2017 17:58
Fishrock123 removed build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. labels Jul 20, 2017
mcollina added 2 commits July 20, 2017 16:18
Fixes: #14386
Fixes: #14381

PR-URL: #14387
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Fixes: #14368

PR-URL: #14387
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
@@ -27,6 +28,17 @@
* [io.js](CHANGELOG_IOJS.md)
* [Archive](CHANGELOG_ARCHIVE.md)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This needs to be updated.

Fishrock123 force-pushed the v8.2.1-proposal branch 2 times, most recently from 9fedddf to 191e812 Compare July 20, 2017 20:53
Notable changes

* http: Writes no longer abort if the Socket is missing.
* process, async_hooks: Avoid problems when triggerAsyncId is undefined.
* zlib: Streams no longer attempt to process data when destroyed.

PR-URL: #14399

Copy link
Copy Markdown
Contributor Author

Fishrock123 merged commit 7e46350 into v8.x Jul 20, 2017
Fishrock123 added a commit that referenced this pull request Jul 20, 2017
Fishrock123 added a commit that referenced this pull request Jul 20, 2017
Notable changes

* http: Writes no longer abort if the Socket is missing.
* process, async_hooks: Avoid problems when triggerAsyncId is undefined.
* zlib: Streams no longer attempt to process data when destroyed.

PR-URL: #14399
Fishrock123 added a commit to nodejs/nodejs.org that referenced this pull request Jul 20, 2017
Fishrock123 deleted the v8.2.1-proposal branch July 20, 2017 22:27

Copy link
Copy Markdown
Contributor Author

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL