Skip to content

benchmark: fix input passed to subtle.digest in webcrypto-digest - #66608

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
RafaelGSS:fix-input-passed-to-subtle-digest-in-webcrypto-dig
Oct 10, 2026
Merged

nodejs-github-bot merged 1 commit into
nodejs:mainfrom
RafaelGSS:fix-input-passed-to-subtle-digest-in-webcrypto-dig

Conversation

@RafaelGSS

Copy link
Copy Markdown
Member

benchmark: fix input passed to subtle.digest in webcrypto-digest

The data is already a random Buffer, so passing it through
TextEncoder decoded it as UTF-8 and re-encoded it, making
subtle.digest hash a different and larger input than createHash.

The data is already a random Buffer, so passing it through
TextEncoder decoded it as UTF-8 and re-encoded it, making
subtle.digest hash a different and larger input than createHash.

Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/performance

@nodejs-github-bot nodejs-github-bot added benchmark Issues and PRs related to Node.js benchmarks and benchmarking infrastructure. crypto Issues and PRs related to the crypto subsystem. labels Oct 8, 2026
@RafaelGSS RafaelGSS added author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. labels Oct 8, 2026
@nodejs-github-bot
nodejs-github-bot merged commit daed8a3 into nodejs:main Oct 10, 2026
48 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in daed8a3

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Oct 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. benchmark Issues and PRs related to Node.js benchmarks and benchmarking infrastructure. crypto Issues and PRs related to the crypto subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants