Skip to content

deps: bump node-forge from 1.4.0 to 1.4.2 - #4025

Closed
eablack wants to merge 1 commit into
mainfrom
eb/fix-node-forge-high
Closed

eablack wants to merge 1 commit into
mainfrom
eb/fix-node-forge-high

Conversation

@eablack

@eablack eablack commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

What

Bumps node-forge from 1.4.0 → 1.4.2 (lockfile-only; root already declares node-forge ^1.3.3).

Why

Resolves the open high-severity Dependabot alert #435 — GHSA-86w9-cpqp-85rv ("node-forge RSA PKCS#1 v1.5 signature verification accepts extra nested DigestAlgorithm elements").

The advisory's vulnerable range is <= 1.4.0. node-forge 1.4.1/1.4.2 were released 2026-10-27, after the advisory was last refreshed (2026-10-01), so GitHub still lists first_patched_version as none — but 1.4.2 is already outside the <= 1.4.0 vulnerable range, so this clears the alert. Dependabot had not opened a PR because its advisory data predates the upstream fix.

Scope

Single transitive copy (node_modules/node-forge), floated via npm update node-forge --package-lock-only. package.json is untouched and the @opentelemetry dependency tree is intact (1675 packages / 66 otel nodes — no dropped-node desync).

Not included (tracked separately)

  • undici (3 high + 2 moderate) and ip-address (1 moderate): the vulnerable copies are bundled inside the vendored npm CLI (pulled in by @oclif/plugin-plugins → npm ^11.18.0). npm@12.2.1 (latest) still bundles undici 6.29.0 / ip-address 10.7.2, and overrides can't reach bundled deps — blocked on npm upstream refreshing its bundle.
  • zod (1 high): only patched version is 4.6.6, a major 3→4 bump; @heroku/mcp-server pins ^3.25.76, so forcing it needs mcp-server compatibility testing — out of scope for a lockfile-only security fix.

Testing

  • CI green (verifies npm ci resolves the updated lockfile and the full test matrix passes)

Resolves the high-severity advisory GHSA-86w9-cpqp-85rv (RSA PKCS#1 v1.5
signature verification accepts extra nested DigestAlgorithm elements),
whose vulnerable range is <= 1.4.0. node-forge 1.4.1/1.4.2 (released
2026-10-27) are outside that range. Root declares node-forge ^1.3.3, so
1.4.2 satisfies; lockfile-only float, otel tree intact (1675/66).
@eablack
eablack requested a review from a team as a code owner October 29, 2026 18:37
@eablack
eablack deployed to AcceptanceTests October 29, 2026 18:37 — with GitHub Actions Active
@eablack
eablack deployed to AcceptanceTests October 29, 2026 18:37 — with GitHub Actions Active
@eablack
eablack deployed to AcceptanceTests October 29, 2026 18:37 — with GitHub Actions Active
@eablack
eablack deployed to AcceptanceTests October 29, 2026 18:37 — with GitHub Actions Active
@github-actions github-actions Bot added the size/XS Extra small PR (code churn < 10) label Oct 29, 2026
@eablack

eablack commented Oct 29, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #4014 (merged 2026-10-29), which bumped @heroku/mcp-server to 1.2.13 and node-forge to 1.4.2 in one change — resolving the node-forge high (GHSA-86w9-cpqp-85rv) alongside the zod high. This standalone node-forge PR is no longer needed. Closing.

@eablack eablack closed this Oct 29, 2026

This branch was successfully deployed

1 active deployment
AcceptanceTests — 5cc5f5a9 Deployed Oct 29, 2026 by eablack via acceptance (22.x, ubuntu-latest) #9587
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR (code churn < 10)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant