Skip to content

fix(website): patch 9 npm security advisories via overrides#474

Merged
szibis merged 1 commit into
mainfrom
fix/website-npm-vulns
Jul 22, 2026
Merged

fix(website): patch 9 npm security advisories via overrides#474
szibis merged 1 commit into
mainfrom
fix/website-npm-vulns

Conversation

@szibis

@szibis szibis commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

What

Clears all 9 open Dependabot alerts in website/ (all transitive) using package.json overrides, plus a js-yaml v4 front-matter engine in docusaurus.config.ts.

Alerts cleared

js-yaml (high, CVE-2026-59869) · ws (high, CVE-2026-48779) · shell-quote (high, CVE-2026-13311) · brace-expansion (CVE-2026-13149) · webpack-dev-server · http-proxy-middleware · joi · body-parser · @babel/core. Also resolves the corresponding website/package-lock.json Trivy findings in the Security workflow.

The js-yaml wrinkle

There is no non-vulnerable js-yaml 3.x, but gray-matter (used by Docusaurus to parse front matter) calls js-yaml v3's safeLoad, removed in v4. So the override to js-yaml 4.3.0 is paired with a markdown.parseFrontMatter shim that drives gray-matter with a js-yaml v4 load engine. The two must ship together.

Verification

  • npm auditfound 0 vulnerabilities
  • npm run typecheck (tsc) → exit 0 ✅
  • npm run build[SUCCESS] Generated static files

Carets pin every package to its current major (no risky major bumps).

@github-actions github-actions Bot added size/XL Extra large change scope/docs Documentation bugfix Bug fix labels Jul 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Quality Report

Compared against base branch main.

Coverage and tests

Signal Base PR Delta
Test count 4792 4792 0
Coverage 85.1% 85.1% 0.0% (stable)

Compatibility

Track Base PR Delta
Loki API 100.0% 11/11 (100.0%) 0.0% (stable)
Logs Drilldown 100.0% 17/17 (100.0%) 0.0% (stable)
VictoriaLogs 100.0% 11/11 (100.0%) 0.0% (stable)

Performance smoke

Performance smoke was skipped for this PR because no perf-sensitive paths changed.

State

  • Coverage, compatibility, and sampled performance are reported here from the same PR workflow.
  • This is a delta report, not a release gate by itself. Required checks still decide merge safety.
  • Performance smoke was intentionally skipped because no perf-sensitive paths changed in this PR.
  • Delta states use the same noise guards as the quality gate (percent + absolute + low-baseline checks), so report labels match merge-gate behavior.

@szibis
szibis force-pushed the fix/website-npm-vulns branch from 120c83c to 5818bbd Compare July 22, 2026 09:17
@github-actions github-actions Bot added size/XL Extra large change and removed size/XL Extra large change labels Jul 22, 2026
Every alert is transitive. Adds package.json "overrides" pinning safe
versions of js-yaml, ws, shell-quote, brace-expansion, webpack-dev-server,
http-proxy-middleware, joi, body-parser and @babel/core, and regenerates
the lockfile (also clearing newer dompurify/fast-uri/svgo advisories via
patch bumps).

js-yaml has no safe 3.x, and gray-matter (Docusaurus front matter) calls
the removed js-yaml v3 safeLoad. So docusaurus.config.ts drives gray-matter
with a js-yaml v4 load engine via markdown.parseFrontMatter.

Verified: npm audit -> 0 vulnerabilities, tsc typecheck ok, site build
succeeds. Docs-site only; no proxy runtime change.
@szibis
szibis force-pushed the fix/website-npm-vulns branch from 5818bbd to cb9d315 Compare July 22, 2026 11:01
@github-actions github-actions Bot added size/XL Extra large change and removed size/XL Extra large change labels Jul 22, 2026
@szibis
szibis merged commit f05fd41 into main Jul 22, 2026
53 checks passed
@szibis
szibis deleted the fix/website-npm-vulns branch July 22, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Bug fix scope/docs Documentation size/XL Extra large change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant