Skip to content

Conversation

@panva
Copy link
Member

@panvapanva commented Jul 5, 2025

Backports #58942 to v20.x

aduh95and others added 2 commits June 23, 2025 12:20
PR-URL: nodejs#58770 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]>
Reverts: nodejs#56160Fixes: nodejs#56159Fixes: nodejs#58913 Refs: nodejs#58121 PR-URL: nodejs#58942 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/crypto

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module. v20.x Issues that can be reproduced on v20.x or PRs targeting the v20.x-staging branch. labels Jul 5, 2025
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/67852/

Copy link

@Hardanish-SinghHardanish-Singh left a comment

Choose a reason for hiding this comment

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

LGTM!

marco-ippolito pushed a commit that referenced this pull request Aug 14, 2025
Reverts: #56160Fixes: #56159Fixes: #58913 Refs: #58121 PR-URL: #58942 Backport-PR-URL: #58961 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
@marco-ippolito
Copy link
Member

Landed in 89366ab

@panvapanva deleted the backport-58942-to-20 branch August 14, 2025 10:44
marco-ippolito pushed a commit that referenced this pull request Aug 25, 2025
Reverts: #56160Fixes: #56159Fixes: #58913 Refs: #58121 PR-URL: #58942 Backport-PR-URL: #58961 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
@panvapanva restored the backport-58942-to-20 branch August 27, 2025 07:20
marco-ippolito pushed a commit that referenced this pull request Aug 27, 2025
Reverts: #56160Fixes: #56159Fixes: #58913 Refs: #58121 PR-URL: #58942 Backport-PR-URL: #58961 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
@panvapanva deleted the backport-58942-to-20 branch August 27, 2025 14:43
marco-ippolito pushed a commit that referenced this pull request Aug 27, 2025
Reverts: #56160Fixes: #56159Fixes: #58913 Refs: #58121 PR-URL: #58942 Backport-PR-URL: #58961 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.cryptoIssues and PRs related to the crypto subsystem.needs-ciPRs that need a full CI run.utilIssues and PRs related to the built-in util module.v20.xIssues that can be reproduced on v20.x or PRs targeting the v20.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@panva@nodejs-github-bot@marco-ippolito@Hardanish-Singh@aduh95