Skip to content

[stable32] Fix npm audit#2507

Merged
artonge merged 2 commits intostable32from
automated/noid/stable32-fix-npm-audit
Apr 14, 2026
Merged

[stable32] Fix npm audit#2507
artonge merged 2 commits intostable32from
automated/noid/stable32-fix-npm-audit

Conversation

@nextcloud-command
Copy link
Copy Markdown
Contributor

@nextcloud-command nextcloud-command commented Apr 12, 2026

Audit report

This audit fix resolves 1 of the total 12 vulnerabilities found in your project.

Updated dependencies

Fixed vulnerabilities

vite #

  • Vite Vulnerable to Path Traversal in Optimized Deps .map Handling
  • Severity: moderate
  • Reference: GHSA-4w7w-66w2-5vf9
  • Affected versions: 7.0.0 - 7.3.1
  • Package usage:
    • node_modules/vite

Additional fixes

Fixed 4 more vulnerabilities (2 high, 1 critical) by adjusting overrides and dependencies:

rollup

  • Rollup 4 has Arbitrary File Write via Path Traversal
  • Severity: high
  • Reference: GHSA-mw96-cpmx-2vgc
  • Affected versions: 4.0.0 - 4.58.0
  • Fix: Removed the rollup: "v4.40.1" override that was pinning vite's rollup to a vulnerable version. Vite 7.3.2 already requires rollup ^4.43.0 which is safe.

happy-dom

  • VM Context Escape can lead to Remote Code Execution
  • Severity: critical
  • Reference: GHSA-37j7-fg3j-429f
  • Affected versions: <=20.8.8
  • Fix: Bumped happy-dom from ^17.5.6 to ^20.8.9.

immutable

  • Immutable is vulnerable to Prototype Pollution
  • Severity: high
  • Reference: GHSA-wf6x-7x77-mvgw
  • Affected versions: 5.0.0 - 5.1.4
  • Fix: Added "immutable": "^5.1.5" override to bump sass's transitive dependency.

Remaining unfixable vulnerabilities (7 low)

elliptic

  • Elliptic Uses a Cryptographic Primitive with a Risky Implementation
  • Severity: low
  • Reference: GHSA-848j-6mx2-7j84
  • Upstream in @nextcloud/vite-configvite-plugin-node-polyfillsnode-stdlib-browsercrypto-browserifyelliptic. No fix available.

Signed-off-by: GitHub <noreply@github.com>
@nextcloud-command nextcloud-command added 3. to review dependencies Pull requests that update a dependency file labels Apr 12, 2026
@cypress
Copy link
Copy Markdown

cypress bot commented Apr 12, 2026

Activity    Run #3578

Run Properties:  status check passed Passed #3578  •  git commit 2508c3e6a9: [stable32] Fix npm audit
Project Activity
Branch Review automated/noid/stable32-fix-npm-audit
Run status status check passed Passed #3578
Run duration 02m 35s
Commit git commit 2508c3e6a9: [stable32] Fix npm audit
Committer Nextcloud Command Bot
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 9
View all changes introduced in this branch ↗︎

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

…able

The rollup v4.40.1 override was pinning vite's rollup to a version
in the vulnerable range (4.0.0-4.58.0, GHSA-mw96-cpmx-2vgc).
Vite 7.3.2 already requires rollup ^4.43.0 which is safe.

Bumped happy-dom from ^17.5.6 to ^20.8.9 to fix critical VM context
escape (GHSA-37j7-fg3j-429f) and other vulnerabilities.

Added immutable override to ^5.1.5 to fix prototype pollution
(GHSA-wf6x-7x77-mvgw) in sass's transitive dependency.

Remaining 7 low-severity elliptic advisories are upstream in
@nextcloud/vite-config via vite-plugin-node-polyfills and cannot
be fixed from this project.

AI-Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala requested review from artonge and susnux April 13, 2026 20:56
@miaulalala miaulalala self-assigned this Apr 13, 2026
@artonge artonge merged commit a71cb1f into stable32 Apr 14, 2026
53 checks passed
@artonge artonge deleted the automated/noid/stable32-fix-npm-audit branch April 14, 2026 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants