crypto: support outputLength option in crypto.hash for XOF functions by Aditi-1400 · Pull Request #58121 · nodejs/node · GitHub

Support outputLength option in crypto.hash() for XOF hash functions to align with the behaviour of crypto.createHash() API

Fixes: #57312

Review requested:

@nodejs/crypto
@nodejs/security-wg

@Aditi-1400 please change the PR title and commit message to crypto: support outputLength in crypto.hash for XOF functions

Codecov Report

Attention: Patch coverage is 82.85714% with 12 lines in your changes missing coverage. Please review.

Project coverage is 90.08%. Comparing base (f5da8f8) to head (a27fe2b).
Report is 68 commits behind head on main.

Files with missing lines	Patch %	Lines
src/crypto/crypto_hash.cc	73.33%	9 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #58121      +/-   ##
==========================================
- Coverage   90.08%   90.08%   -0.01%     
==========================================
  Files         640      640              
  Lines      188517   188569      +52     
  Branches    36975    36995      +20     
==========================================
+ Hits       169826   169868      +42     
- Misses      11402    11418      +16     
+ Partials     7289     7283       -6

Files with missing lines	Coverage Δ
lib/internal/crypto/hash.js	98.53% <100.00%> (+0.09%)	⬆️
src/crypto/crypto_hash.cc	70.35% <73.33%> (-0.21%)	⬇️

... and 29 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codebytere ... can I ask you to take a look at this from a Whether-This-Will-Work-In-BoringSSL perspective?

Reverts: nodejs#56160 Fixes: nodejs#56159 Fixes: nodejs#58913 Refs: nodejs#58121

Needs a new CI after the push

@joyeecheung CI has ran with the latest push.

Landed in 1c4fe6d

Reverts: #56160 Fixes: #56159 Fixes: #58913 Refs: #58121 PR-URL: #58942 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Support `outputLength` option in crypto.hash() for XOF hash functions to align with the behaviour of crypto.createHash() API closes: #57312 Co-authored-by: Filip Skokan <panva.ip@gmail.com> PR-URL: #58121 Fixes: #57312 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

Notable changes: crypto: * (SEMVER-MINOR) support outputLength option in crypto.hash for XOF functions (Aditi) #58121 doc: * (SEMVER-MINOR) add all watch-mode related flags to node.1 (Dario Piotrowicz) #58719 fs: * (SEMVER-MINOR) add disposable mkdtempSync (Kevin Gibbons) #58516 permission: * (SEMVER-MINOR) propagate permission model flags on spawn (Rafael Gonzaga) #58853 sqlite: * (SEMVER-MINOR) add support for readBigInts option in db connection level (Miguel Marcondes Filho) #58697 src,permission: * (SEMVER-MINOR) add support to permission.has(addon) (Rafael Gonzaga) #58951 watch: * (SEMVER-MINOR) add `--watch-kill-signal` flag (Dario Piotrowicz) #58719 PR-URL: #58993

Reverts: nodejs#56160 Fixes: nodejs#56159 Fixes: nodejs#58913 Refs: nodejs#58121 PR-URL: nodejs#58942 Backport-PR-URL: nodejs#58960 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

This doesn't land cleanly on v22.x-staging

Reverts: #56160 Fixes: #56159 Fixes: #58913 Refs: #58121 PR-URL: #58942 Backport-PR-URL: #58961 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Support `outputLength` option in crypto.hash() for XOF hash functions to align with the behaviour of crypto.createHash() API closes: nodejs/node#57312 Co-authored-by: Filip Skokan <panva.ip@gmail.com> PR-URL: nodejs/node#58121 Fixes: nodejs/node#57312 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

This pull request at least depends on #56653, #57300 and #54028

Support `outputLength` option in crypto.hash() for XOF hash functions to align with the behaviour of crypto.createHash() API closes: nodejs/node#57312 Co-authored-by: Filip Skokan <panva.ip@gmail.com> PR-URL: nodejs/node#58121 Fixes: nodejs/node#57312 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>

nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels May 2, 2025

panva requested review from joyeecheung and tniessen May 2, 2025 14:26

panva added the crypto Issues and PRs related to the crypto subsystem. label May 2, 2025

joyeecheung reviewed May 2, 2025

View reviewed changes

Comment thread deps/ncrypto/ncrypto.cc

panva reviewed May 2, 2025

View reviewed changes

Aditi-1400 changed the title src: support outputLength in XOF hash functions crypto: support outputLength in XOF hash functions May 2, 2025

jasnell reviewed May 6, 2025

View reviewed changes

Comment thread deps/ncrypto/ncrypto.cc Outdated

jasnell reviewed May 6, 2025

View reviewed changes

Comment thread deps/ncrypto/ncrypto.h

jasnell reviewed May 6, 2025

View reviewed changes

Comment thread doc/api/crypto.md Outdated

Aditi-1400 force-pushed the crypto-hash branch from d0594f4 to aeab00b Compare May 12, 2025 17:12

Aditi-1400 force-pushed the crypto-hash branch from aeab00b to f40c837 Compare May 27, 2025 12:50

joyeecheung reviewed May 28, 2025

View reviewed changes

Comment thread src/crypto/crypto_hash.cc Outdated

Aditi-1400 force-pushed the crypto-hash branch from f40c837 to b9379c5 Compare May 28, 2025 21:07

panva mentioned this pull request Jul 3, 2025

crypto: fix SHAKE128/256 breaking change introduced with OpenSSL 3.4 #58942

Merged

joyeecheung approved these changes Jul 3, 2025

View reviewed changes

Comment thread doc/api/crypto.md Outdated

Aditi-1400 force-pushed the crypto-hash branch from b9379c5 to c44c09e Compare July 3, 2025 12:37

joyeecheung reviewed Jul 3, 2025

View reviewed changes

Comment thread deps/ncrypto/ncrypto.cc Outdated

panva added a commit to panva/node that referenced this pull request Jul 3, 2025

crypto: fix SHAKE128/256 breaking change introduced with OpenSSL 3.4

d870b4f

Reverts: nodejs#56160 Fixes: nodejs#56159 Fixes: nodejs#58913 Refs: nodejs#58121

Aditi-1400 force-pushed the crypto-hash branch 3 times, most recently from beb4f68 to 068c2ce Compare July 3, 2025 17:26

joyeecheung added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 8, 2025

panva removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 8, 2025

panva added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 8, 2025

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 8, 2025

nodejs-github-bot merged commit 1c4fe6d into nodejs:main Jul 8, 2025

github-actions Bot mentioned this pull request Jul 8, 2025

2025-07-09, Version 24.4.0 (Current) #58993

Merged

Renegade334 mentioned this pull request Jul 14, 2025

Node 24.4.0 OOMs with pnpm #59057

Closed

aduh95 added the backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. label Jul 21, 2025

richardlau mentioned this pull request Sep 19, 2025

crypto: avoid copying buffers to UTF-8 strings in crypto.hash() #59067

Closed

+              DataPointer xofHashDigest(const Buffer<const unsigned char>& buf,
+                                        const EVP_MD* md,
+                                        size_t output_length) {
+                if (md == nullptr) return {};

crypto: support outputLength option in crypto.hash for XOF functions by Aditi-1400 · Pull Request #58121 · nodejs/node · GitHub

Uh oh!

Uh oh!

crypto: support outputLength option in crypto.hash for XOF functions - #58121

crypto: support outputLength option in crypto.hash for XOF functions#58121nodejs-github-bot merged 1 commit intonodejs:mainnodejs/node:mainfrom Aditi-1400:crypto-hashAditi-1400/node:crypto-hashCopy head branch name to clipboard

Conversation

Aditi-1400 commented May 2, 2025

Uh oh!

nodejs-github-bot commented May 2, 2025

Uh oh!

Uh oh!

panva left a comment

Choose a reason for hiding this comment

Uh oh!

codecov Bot commented May 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

jasnell May 6, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

joyeecheung commented Jul 8, 2025

Uh oh!

panva commented Jul 8, 2025

Uh oh!

nodejs-github-bot commented Jul 8, 2025

Uh oh!

aduh95 commented Jul 21, 2025

Uh oh!

Aditi-1400 commented Sep 24, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

crypto: support outputLength option in crypto.hash for XOF functions#58121
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
Aditi-1400:crypto-hash

codecov Bot commented May 2, 2025 •
edited
Loading