Skip to content

[TOOL-5898] Update tar to >=7.5.4 to resolve CVE-2026-23950#14

Merged
ThomasTrepanier merged 1 commit intomainfrom
thomtrep/TOOL-5898-tar
Feb 24, 2026
Merged

[TOOL-5898] Update tar to >=7.5.4 to resolve CVE-2026-23950#14
ThomasTrepanier merged 1 commit intomainfrom
thomtrep/TOOL-5898-tar

Conversation

@ThomasTrepanier
Copy link

@ThomasTrepanier ThomasTrepanier commented Feb 23, 2026

This change is Reviewable


Summary

  • Added resolutions in package.json to force tar@>=7.5.4, resolving CVE-2026-23950 (high severity race condition in node-tar path reservations via Unicode collisions on macOS APFS).
  • Upstream @yarnpkg/core (latest 4.5.0) still depends on tar@^6 with no patched v6 release available, so a resolution override was the only viable fix path.
  • All tar instances now resolve to 7.5.9.
  • Jira: https://maintainx.atlassian.net/browse/TOOL-5898

@ThomasTrepanier ThomasTrepanier marked this pull request as ready for review February 23, 2026 22:38
@ThomasTrepanier ThomasTrepanier requested review from a team, BobbyWhiskey and cptCloud and removed request for a team February 23, 2026 22:38
Added resolutions to force tar>=7.5.4 since upstream @yarnpkg/core
still requires tar@^6 with no patched v6 release available.
@ThomasTrepanier ThomasTrepanier enabled auto-merge (squash) February 23, 2026 22:39
Copy link

@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.

},
"resolutions": {
"tar": "^7.5.4"
},
Copy link

Choose a reason for hiding this comment

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

Untested tar v7 compatibility with Yarn internals

Medium Severity

Forcing tar@^7.5.4 via resolutions upgrades @yarnpkg/core from its expected tar@^6 dependency to v7, which includes a breaking change: the chmod option now defaults to false instead of implicitly true. This means extracted files won't preserve permissions from tar archives by default. While this likely won't affect normal package extraction (system defaults apply), it's an untested configuration that could cause unexpected behavior with executable files or special permissions if Yarn's internal code relies on tar v6's permission-preserving behavior.

Fix in Cursor Fix in Web

Copy link

@BobbyWhiskey BobbyWhiskey left a comment

Choose a reason for hiding this comment

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

@BobbyWhiskey reviewed 2 files and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on cptCloud and ThomasTrepanier).

@ThomasTrepanier ThomasTrepanier merged commit 17c8b9f into main Feb 24, 2026
2 of 3 checks passed
@ThomasTrepanier ThomasTrepanier deleted the thomtrep/TOOL-5898-tar branch February 24, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants