Skip to content

OUT-3587: pin axios to ^1.15.2 via yarn resolutions - #234

Merged
SandipBajracharya merged 1 commit into
masterfrom
OUT-3587-re
Apr 22, 2026
Merged

OUT-3587: pin axios to ^1.15.2 via yarn resolutions#234
SandipBajracharya merged 1 commit into
masterfrom
OUT-3587-re

Conversation

@SandipBajracharya

@SandipBajracharya SandipBajracharya commented Apr 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Resolves Dependabot alerts for axios <1.15.0 pulled transitively through intuit-oauth@4.2.3 (which pins axios@1.14.0).
  • Affected advisories: GHSA-3p68-rc4w-qgx5 (NO_PROXY SSRF) and GHSA-fvcv-3m26-pcqx (cloud metadata exfiltration).
  • intuit-oauth is already at its latest version (4.2.3) with no upstream fix available. The yarn resolutions override is SemVer-compatible with its declared axios: ^1.9.0 range, so no API drift.

Linear: OUT-3587

Test plan

  • yarn install — axios resolves to 1.15.2 (verified via yarn why axios)
  • yarn npm audit --all --recursive — both advisories no longer reported
  • yarn tsc --noEmit — no new type errors introduced by the bump
  • Smoke test QuickBooks OAuth flow in staging (intuit-oauth is the only consumer of axios)

🤖 Generated with Claude Code

Addresses Dependabot alerts GHSA-3p68-rc4w-qgx5 (NO_PROXY SSRF) and
GHSA-fvcv-3m26-pcqx (cloud metadata exfiltration) pulled transitively
through intuit-oauth@4.2.3 (which pins axios@1.14.0). intuit-oauth is
at its latest version with no upstream fix available; the resolution
override is SemVer-compatible with its declared ^1.9.0 range.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@linear-code

linear-code Bot commented Apr 22, 2026

Copy link
Copy Markdown

@vercel

vercel Bot commented Apr 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
quickbooks-sync Building Building Apr 22, 2026 9:34am
quickbooks-sync (dev) Ready Ready Preview, Comment Apr 22, 2026 9:34am

Request Review

@SandipBajracharya SandipBajracharya changed the title fix(OUT-3587): pin axios to ^1.15.2 via yarn resolutions OUT-3587: pin axios to ^1.15.2 via yarn resolutions Apr 22, 2026
@greptile-apps

greptile-apps Bot commented Apr 22, 2026

Copy link
Copy Markdown

Greptile Summary

This PR addresses two critical axios security advisories (SSRF via NO_PROXY bypass and cloud metadata exfiltration) that were pulled transitively through intuit-oauth@4.2.3, which hardcodes axios@1.14.0. Since no upstream fix is available in intuit-oauth, a Yarn resolutions override pinning axios to ^1.15.2 is the correct mitigation strategy. The lockfile confirms intuit-oauth's ^1.9.0 request is remapped to the single ^1.15.2 entry, and the ^1.15.2 range is fully SemVer-compatible with intuit-oauth's declared ^1.9.0 range.

Confidence Score: 5/5

Safe to merge — minimal, targeted change that resolves two CVEs with no API drift or breaking changes.

The change is a one-line addition to resolutions plus the corresponding lockfile update. The ^1.15.2 range is SemVer-compatible with intuit-oauth's ^1.9.0 declaration, the lockfile correctly remaps the transitive request, and no other consumers of axios exist in this repo. All remaining considerations are P2 or lower.

No files require special attention.

Important Files Changed

Filename Overview
package.json Adds "axios": "^1.15.2" to the Yarn resolutions field to force the transitive axios version above the vulnerable 1.14.0 pinned by intuit-oauth
yarn.lock Replaces the single axios lockfile entry from ^1.9.0 / 1.14.0 to ^1.15.2 / 1.15.2 with updated checksum; intuit-oauth's ^1.9.0 request is correctly remapped to this entry by Yarn's resolution override

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["intuit-oauth@4.2.3\ndeclares axios: ^1.9.0"] --> B{Yarn resolution\noverride in package.json}
    B -- "Before PR\n(no override)" --> C["axios@1.14.0\n❌ GHSA-3p68-rc4w-qgx5\n❌ GHSA-fvcv-3m26-pcqx"]
    B -- "After PR\n(axios: ^1.15.2)" --> D["axios@1.15.2\n✅ NO_PROXY bypass fixed\n✅ metadata exfiltration fixed"]
    D --> E["yarn.lock entry key\naxios@npm:^1.15.2\nremaps intuit-oauth's ^1.9.0 request"]
Loading

Reviews (1): Last reviewed commit: "fix(OUT-3587): pin axios to ^1.15.2 via ..." | Re-trigger Greptile

@SandipBajracharya
SandipBajracharya merged commit d6210d7 into master Apr 22, 2026
4 checks passed
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