Skip to content

Site: clear the new npm audit advisories - #499

Merged
fstubner merged 1 commit into
mainfrom
fix/site-audit-basic-ftp
Oct 2, 2026
Merged

fstubner merged 1 commit into
mainfrom
fix/site-audit-basic-ftp

Conversation

@fstubner

@fstubner fstubner commented Oct 2, 2026

Copy link
Copy Markdown
Owner

The site's npm audit check started failing on every PR today with a new advisory for basic-ftp (GHSA-c475-qrg2-pj4r, high). It reaches the site only through the accessibility and Lighthouse tooling. get-uri pins ^5.3.1 and has no newer release, so this overrides basic-ftp to ^6.2.1. npm audit fix also takes patch releases of devalue (5.9.2 → 5.9.4) and fast-uri (3.1.7 → 3.1.8).

Local npm audit: 0 vulnerabilities. The site build is left to the Site Gate here, since this worktree has no node_modules.

basic-ftp <=6.2.0 has a quadratic-time parser (GHSA-c475-qrg2-pj4r). It
reaches the site only through the accessibility and Lighthouse tooling
(get-uri pins ^5.3.1 and has no newer release), so override it to ^6.2.1.
devalue and fast-uri take patch releases from `npm audit fix`.
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

Site preview: https://pr-499.netscli-site-preview.pages.dev

Built from 29913c2 with NETSCLI_PREVIEW=1 — noindex, and analytics disabled so it does not report into netscli.com's numbers.

Production is unaffected: netscli.com is served from GitHub Pages via pages.yml, which is manual-only.

@fstubner
fstubner merged commit b630707 into main Oct 2, 2026
17 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.

1 participant