Skip to content

crypto: runtime-deprecate calling Hmac.digest() more than once#63624

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
developers-universe-1:dep0206-hmac-digest-runtime-deprecation
Jun 15, 2026
Merged

crypto: runtime-deprecate calling Hmac.digest() more than once#63624
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
developers-universe-1:dep0206-hmac-digest-runtime-deprecation

Conversation

@developers-universe-1

Copy link
Copy Markdown
Contributor

Emit a DEP0206 deprecation warning when Hmac.digest() is called on an already-finalized instance. This behavior is inconsistent with Hash.digest() which throws ERR_CRYPTO_HASH_FINALIZED.

Fixes: #62838

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto
  • @nodejs/userland-migrations

@nodejs-github-bot nodejs-github-bot added crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. labels May 28, 2026
@panva panva added deprecations Issues and PRs related to deprecations. semver-major PRs that contain breaking changes and should be released in the next major version. labels Jun 15, 2026
Emit a DEP0206 deprecation warning when Hmac.digest() is called on an
already-finalized instance. This behavior is inconsistent with
Hash.digest() which throws ERR_CRYPTO_HASH_FINALIZED.

Fixes: nodejs#62838
Co-authored-by: Filip Skokan <panva.ip@gmail.com>
Signed-off-by: developers-universe-1 <madelynreyes2026@gmail.com>
Signed-off-by: Filip Skokan <panva.ip@gmail.com>
@panva panva force-pushed the dep0206-hmac-digest-runtime-deprecation branch from 59c5a36 to 707d1dd Compare June 15, 2026 07:47
@panva panva added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 15, 2026
@panva panva requested a review from ChALkeR June 15, 2026 08:48
@panva

panva commented Jun 15, 2026

Copy link
Copy Markdown
Member

cc @nodejs/tsc for semver-major PRs that contain breaking changes and should be released in the next major version.

@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 15, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@panva panva added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 15, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 15, 2026
@nodejs-github-bot nodejs-github-bot merged commit 463e56f into nodejs:main Jun 15, 2026
73 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 463e56f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. crypto Issues and PRs related to the crypto subsystem. deprecations Issues and PRs related to deprecations. needs-ci PRs that need a full CI run. semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Runtime-deprecate calling digest() on HMAC more than once

6 participants