Skip to content

Conversation

@RafaelGSS
Copy link
Member

2024-01-22, Version 21.6.1 (Current), @RafaelGSS

Notable Changes

This release fixes a bug in undici using WebStreams

Commits

  • [662ac95729] - Revert "stream: fix cloned webstreams not being unref'd" (Matteo Collina) #51491
  • [1b8bba8aee] - test: add regression test for 51586 (Matteo Collina) #51491

This reverts commit 4d3923a. PR-URL: #51491 Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Matthew Aitken <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
Signed-off-by: Matteo Collina <[email protected]> PR-URL: #51491 Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Matthew Aitken <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
@nodejs-github-botnodejs-github-bot added meta Issues and PRs related to the general management of the project. needs-ci PRs that need a full CI run. v21.x web streams labels Jan 19, 2024
RafaelGSS added a commit that referenced this pull request Jan 19, 2024
Notable changes: This release fixes a bug in `undici` using WebStreams PR-URL: #51530
@RafaelGSS
Copy link
MemberAuthor

RafaelGSS commented Jan 19, 2024

Considering I won't release it until Monday 22/01/2024 if #51526 gets landed on this, I can include it in this patch.

cc: @mcollina

@RafaelGSSRafaelGSS added release Issues and PRs related to Node.js releases. and removed meta Issues and PRs related to the general management of the project. web streams labels Jan 19, 2024
Copy link
Member

@mcollinamcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@RafaelGSSRafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 20, 2024
Notable changes: This release fixes a bug in `undici` using WebStreams PR-URL: #51530
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 20, 2024
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jan 20, 2024

@RafaelGSS
Copy link
MemberAuthor

@anonrig can you dismiss the review?

@RafaelGSS
Copy link
MemberAuthor

FWIW Release is blocked because we are facing a problem with macOS binaries.

RafaelGSS added a commit that referenced this pull request Jan 22, 2024
@RafaelGSSRafaelGSS merged commit 3dc0e49 into v21.xJan 22, 2024
@RafaelGSSRafaelGSS deleted the v21.6.1-proposal branch January 22, 2024 19:16
RafaelGSS added a commit that referenced this pull request Jan 22, 2024
Notable changes: This release fixes a bug in `undici` using WebStreams PR-URL: #51530
RafaelGSS added a commit to RafaelGSS/nodejs.org that referenced this pull request Jan 22, 2024
github-merge-queuebot pushed a commit to nodejs/nodejs.org that referenced this pull request Jan 22, 2024
marco-ippolito pushed a commit to marco-ippolito/node that referenced this pull request Feb 2, 2024
Notable changes: This release fixes a bug in `undici` using WebStreams PR-URL: nodejs#51530
marco-ippolito pushed a commit to marco-ippolito/node that referenced this pull request Feb 19, 2024
Notable changes: This release fixes a bug in `undici` using WebStreams PR-URL: nodejs#51530
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.releaseIssues and PRs related to Node.js releases.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RafaelGSS@nodejs-github-bot@mcollina@anonrig