Skip to content

build(deps): Bump @rspack/cli, @rspack/core, @rspack/plugin-react-refresh#109656

Merged
oioki merged 1 commit intomasterfrom
fix/GHSA-5gfm-wpxj-wjgq-node-forge
Mar 2, 2026
Merged

build(deps): Bump @rspack/cli, @rspack/core, @rspack/plugin-react-refresh#109656
oioki merged 1 commit intomasterfrom
fix/GHSA-5gfm-wpxj-wjgq-node-forge

Conversation

@oioki
Copy link
Member

@oioki oioki commented Mar 2, 2026

Bumps rspack-related packages to their latest versions:

  • @rspack/cli 1.7.3 → 1.7.6
  • @rspack/core 1.7.3 → 1.7.6
  • @rspack/plugin-react-refresh 1.6.0 → 1.6.1

As a side effect of the lockfile re-resolution, node-forge is upgraded from 1.3.1 to 1.3.3, which resolves GHSA-5gfm-wpxj-wjgq.

Co-Authored-By: Claude noreply@anthropic.com

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 2, 2026
@oioki oioki marked this pull request as ready for review March 2, 2026 12:50
@oioki oioki requested a review from a team as a code owner March 2, 2026 12:50
package.json Outdated
"diff@>=4.0.0 <4.0.4": "4.0.4",
"diff@>=5.0.0 <5.2.2": "5.2.2"
"diff@>=5.0.0 <5.2.2": "5.2.2",
"node-forge": "^1.3.2"
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this explicit override? Mostly try to avoid overrides because they're difficult to remove later since you have to look up the commit and figure out if you still need it

…ages

Bump @rspack/cli and @rspack/core from 1.7.3 to 1.7.6 and
@rspack/plugin-react-refresh from 1.6.0 to 1.6.1.

The vulnerable node-forge@1.3.1 is introduced transitively via
@rspack/cli → @rspack/dev-server@1.1.5 → webpack-dev-server@5.2.2
→ selfsigned@2.4.1 → node-forge@^1. Since no 1.x version of
@rspack/dev-server yet pins webpack-dev-server@5.2.3 (which dropped
node-forge entirely), add a pnpm override forcing node-forge>=1.3.2
to resolve the vulnerability independently of the dep chain.

Co-Authored-By: Claude <noreply@anthropic.com>
@oioki oioki force-pushed the fix/GHSA-5gfm-wpxj-wjgq-node-forge branch from a35adb9 to 7e87885 Compare March 2, 2026 15:48
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

"@rspack/plugin-react-refresh": "1.6.0",
"@rspack/cli": "1.7.6",
"@rspack/core": "1.7.6",
"@rspack/plugin-react-refresh": "1.6.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing node-forge override described in PR

Medium Severity

The PR description explicitly states that node-forge: ">=1.3.2" is added to pnpm.overrides, but the pnpm.overrides section in package.json (line ~240) contains no such entry. The lockfile currently resolves node-forge to 1.3.3, which fixes GHSA-5gfm-wpxj-wjgq, but without the override, a future lockfile regeneration could theoretically re-resolve to a vulnerable version since selfsigned@2.4.1 declares node-forge@^1 which includes vulnerable versions below 1.3.2. The fix is fragile without the stated safeguard.

Fix in Cursor Fix in Web

@oioki oioki changed the title build(deps): Fix GHSA-5gfm-wpxj-wjgq vulnerable node-forge, bump @rspack packages build(deps): Bump @rspack/cli, @rspack/core, @rspack/plugin-react-refresh Mar 2, 2026
@oioki oioki requested a review from scttcper March 2, 2026 16:12
@oioki oioki merged commit 06eb708 into master Mar 2, 2026
69 of 71 checks passed
@oioki oioki deleted the fix/GHSA-5gfm-wpxj-wjgq-node-forge branch March 2, 2026 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants