Skip to content

fix: upgrade tar to 7.5.19 (CVE-2026-59873)#6068

Open
anupamme wants to merge 2 commits into
nexu-io:mainfrom
anupamme:fix-repo-open-design-cve-2026-59873-tar
Open

fix: upgrade tar to 7.5.19 (CVE-2026-59873)#6068
anupamme wants to merge 2 commits into
nexu-io:mainfrom
anupamme:fix-repo-open-design-cve-2026-59873-tar

Conversation

@anupamme

@anupamme anupamme commented Jul 25, 2026

Copy link
Copy Markdown

Summary

Upgrade tar from 7.5.15 to 7.5.19 to fix CVE-2026-59873.

Vulnerability

Field Value
ID CVE-2026-59873
Severity CRITICAL
Scanner trivy
Rule CVE-2026-59873
File pnpm-lock.yaml
Assessment Likely exploitable

Description: tar: node-tar: Denial of Service via crafted gzip bomb

Evidence

Scanner confirmation: trivy rule CVE-2026-59873 flagged this pattern.

Production code: This file is in the production codebase, not test-only code.

Threat Model Context

This is a Node.js library - vulnerabilities affect downstream consumers who use this package.

Surface area: None

Changes

  • package.json
  • pnpm-lock.yaml

Behavior Preservation

The change is scoped to 2 files on the vulnerable path, and the project's existing tests still pass, so intended behavior is unchanged.

Verification

  • Build passes
  • Scanner re-scan confirms fix
  • Ran pnpm install to regenerate the lockfile β€” tar@7.5.19 is still resolved everywhere.

This change addresses a pattern flagged by static analysis. The code path handles user-influenced input and the fix reduces the attack surface against both manual and automated exploitation.


Automated security fix by OrbisAI Security

Automated dependency upgrade by OrbisAI Security
@lefarcen

Copy link
Copy Markdown
Contributor

The security intent is clear from the summary and verification section. Could you add the Surface area checklist and mark None if this is dependency-only? That helps pool review confirm there isn't any user-facing impact hidden behind the lockfile update.

@lefarcen
lefarcen requested a review from PerishCode July 25, 2026 04:20
@lefarcen lefarcen added size/XS PR changes <20 lines risk/high High risk: apps/desktop, daemon, auth, migration, workflows, package deps type/bugfix Bug fix labels Jul 25, 2026
@anupamme

Copy link
Copy Markdown
Author

Surface area: None. This only bumps tar 7.5.15 β†’ 7.5.19 to fix CVE-2026-59873 (a DoS via unbounded gzip decompression). No application code changes, no config changes, no new files touched besides the manifest/lockfile.

tar's own release notes show this as a bugfix/hardening release (adds limits on decompressed size/entry count), not a breaking API change, so no behaviour change is expected on the happy path.

@lefarcen

Copy link
Copy Markdown
Contributor

Thanks β€” that clarifies the scope. Could you copy two bits into the PR description itself so pool review has everything in one place: Surface area: None, and a short bug-fix verification note that this is a dependency-only bump to tar 7.5.19 with no expected happy-path behavior change?

tar is not a direct open-design import; using pnpm.overrides keeps the
7.5.19 version forced across the transitive tree without adding an
unused top-level dependency to the manifest.
@anupamme

Copy link
Copy Markdown
Author

Addressed the dependency-promotion issue: tar isn't imported directly by open-design, so adding it to dependencies was the wrong fix vector. Switched to pnpm.overrides (matching the existing pattern for brace-expansion, postcss, qs, etc.), which forces the resolved version across the transitive tree without introducing an unused direct dependency into the manifest. Ran pnpm install to regenerate the lockfile; tar@7.5.19 is still resolved everywhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk/high High risk: apps/desktop, daemon, auth, migration, workflows, package deps size/XS PR changes <20 lines type/bugfix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants