Skip to content

fix(deps): tar 7.5.22 for project transfer archives - #30

Merged
sheehanmunim merged 1 commit into
mainfrom
chore/bump-tar
Sep 19, 2026
Merged

sheehanmunim merged 1 commit into
mainfrom
chore/bump-tar

Conversation

@sheehanmunim

Copy link
Copy Markdown
Member

Dependabot (just enabled on this repo) flagged tar 7.5.16 in apps/server: one critical and several high denial-of-service advisories for hostile archives, all fixed by 7.5.21.

tar is used only by the fork's own ProjectTransferFiles (not in upstream T3 Code). It extracts project archives that arrive from another machine, so a crafted archive could hang or crash the server. This bumps it to 7.5.22; node-gyp's copy (build-time, ^7.5.4) moves to 7.5.22 as well.

  • src/project/ProjectTransferFiles.test.ts: 6/6 pass on 7.5.22.
  • Lockfile-only change otherwise; no upstream conflict expected.

The other 16 open alerts outside .repos/ are transitive (image-size has no fix; path-to-regexp, decode-uri-component, uuid, valibot, xml2js) and are not addressed here. 126 more are in the vendored .repos/ reference checkouts, which do not ship.

🤖 Generated with Claude Code

node-tar 7.5.16 has denial-of-service bugs when parsing hostile archives
(unbounded decompression, negative entry sizes, NUL bytes and numeric
PAX paths, uncontrolled recursion; GHSA critical/high, fixed by 7.5.21).
ProjectTransferFiles extracts archives received from another machine,
so it takes the patched release. node-gyp's copy moves to 7.5.22 too.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 19, 2026 18:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Sep 19, 2026
@sheehanmunim
sheehanmunim merged commit 05b74c8 into main Sep 19, 2026
7 of 16 checks passed
@sheehanmunim
sheehanmunim deleted the chore/bump-tar branch September 19, 2026 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants