Skip to content

[CLNP-8635] SECURE-3288: remove unused axios and perspective-api-client (SCA fix)#1

Open
sf-tyler-jeong wants to merge 1 commit into
mainfrom
fix/clnp-8635
Open

[CLNP-8635] SECURE-3288: remove unused axios and perspective-api-client (SCA fix)#1
sf-tyler-jeong wants to merge 1 commit into
mainfrom
fix/clnp-8635

Conversation

@sf-tyler-jeong

Copy link
Copy Markdown
Collaborator

Resolves the two critical axios advisories tracked in SECURE-3288:

Both were unused dependencies, so they are removed rather than bumped:

  • axios is never imported (index.js; googleapis uses its own gaxios client), so removing it eliminates the vulnerable package entirely instead of re-vendoring a bumped-but-unused copy.
  • perspective-api-client pinned a vulnerable nested axios 0.18.1 and is unused (index.js calls the Perspective API via googleapis). README install step updated to match.

package-lock.json and node_modules updated accordingly; node_modules/.package-lock.json (npm's hidden state file) is no longer tracked. npm audit reports zero axios advisories.

Resolves the two critical axios advisories tracked in SECURE-3288:
- CVE-2025-62718 / GHSA-3p68-rc4w-qgx5 (NO_PROXY hostname bypass -> SSRF)
- CVE-2026-40175 / GHSA-fvcv-3m26-pcqx (header injection -> SSRF)

Both were unused dependencies, so they are removed rather than bumped:
- axios is never imported (index.js; googleapis uses its own gaxios client), so removing it eliminates the vulnerable package entirely instead of re-vendoring a bumped-but-unused copy.
- perspective-api-client pinned a vulnerable nested axios 0.18.1 and is unused (index.js calls the Perspective API via googleapis). README install step updated to match.

package-lock.json and node_modules updated accordingly; node_modules/.package-lock.json (npm's hidden state file) is no longer tracked. npm audit reports zero axios advisories.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant