fix: upgrade Next.js to 16.2.12 and pin patched tar (npm audit, #214) - #235
Open
Golchic wants to merge 1 commit into
Open
fix: upgrade Next.js to 16.2.12 and pin patched tar (npm audit, #214)#235Golchic wants to merge 1 commit into
Golchic wants to merge 1 commit into
Conversation
…t advisories Addresses the critical and runtime-exploitable advisories reported in agentsmd#214. - next 16.1.0 -> 16.2.12: clears every Next.js advisory flagged by `pnpm audit` (SSRF in Server Actions and rewrites, App Router / Pages middleware & proxy bypass, multiple DoS vectors, beforeInteractive XSS). Stays on the 16.2.x line referenced in the issue. `pnpm build` verified (all 3 pages generated). - Add pnpm.overrides "tar@<7.5.19" -> ">=7.5.19". @tailwindcss/oxide floats tar at ^7.4.3 but the lockfile pinned 7.4.3, which carries a critical decompression DoS (GHSA-23hp-3jrh-7fpw) plus several hardlink/symlink path-traversal advisories. Override resolves tar 7.5.22. - Regenerate pnpm-lock.yaml. This migrates lockfileVersion 6.0 -> 9.0; the repo already pins packageManager pnpm@9.15.1, so the committed 6.0 lockfile was stale and any `pnpm install` regenerates it as 9.0. - next-env.d.ts: regenerated by Next 16.2 (routes.d.ts path change). pnpm audit: 62 vulnerabilities (1 critical / 33 high / 24 moderate / 4 low) -> 19 (0 critical / 12 high / 6 moderate / 1 low). The remaining 19 are all build-time-only transitive deps under @svgr/webpack (js-yaml, svgo), styled-jsx (postcss) and next (sharp). This site is statically exported, so none are in the deployed runtime, and clearing them would require forced major-version overrides on shared build tooling - out of scope for this security fix. Refs agentsmd#214
|
@Golchic is attempting to deploy a commit to the openai Team on Vercel. A member of the Team first needs to authorize it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Addresses the critical and runtime-exploitable advisories from #214.
next16.1.0→16.2.12pnpm auditreports — SSRF in Server Actions & rewrites, App Router / Pages middleware & proxy bypass, multiple DoS vectors,beforeInteractiveXSS. Stays on the16.2.xline noted in the issue.pnpm.overrides"tar@<7.5.19"→">=7.5.19"@tailwindcss/oxidefloatstarat^7.4.3, but the lockfile pinned7.4.3, which carries a critical decompression DoS (GHSA-23hp-3jrh-7fpw) plus several hardlink/symlink path-traversal advisories. Override resolvestar@7.5.22.pnpm-lock.yamllockfileVersion6.0→9.0. The repo already pinspackageManager: pnpm@9.15.1, so the committed6.0lockfile was stale — anypnpm installregenerates it as9.0. This is why the lockfile diff is large.next-env.d.tsroutes.d.tspath change).Result
pnpm audit: 62 vulnerabilities (1 critical / 33 high / 24 moderate / 4 low) → 19 (0 critical / 12 high / 6 moderate / 1 low).The remaining 19 are all build-time-only transitive deps:
js-yaml/svgounder@svgr/webpack,postcsspinned insidestyled-jsx, andsharpundernext. This site is statically exported, so none are in the deployed runtime. Clearing them would require forced major-version overrides on shared build tooling (andnanoidcan't be bumped without breakingpostcss@8), so they're left out of this security-focused PR.Verification
pnpm install— cleanpnpm build— passes, all 3 pages generated (Next.js 16.2.12, Turbopack)Fixes #214