Skip to content

chore(deps): patch transitive vulns in docs and bump requests dev floor#998

Open
petercrocker wants to merge 1 commit intostablefrom
shining-freeze
Open

chore(deps): patch transitive vulns in docs and bump requests dev floor#998
petercrocker wants to merge 1 commit intostablefrom
shining-freeze

Conversation

@petercrocker
Copy link
Copy Markdown
Contributor

Summary

  • Adds an overrides block to docs/package.json pinning patched versions of 12 vulnerable transitive deps pulled in by Docusaurus, and regenerates docs/package-lock.json against the current npm registry.
  • Bumps the requests dev-dependency floor to >=2.33.0 to clear GHSA-gc5v-m9x4-r6x2 (insecure temp file reuse in extract_zipped_paths).

Why now (vs. waiting for Docusaurus)

26 of the 27 open Dependabot alerts are in docs/package-lock.json — all transitive deps of Docusaurus, none declared directly in docs/package.json. @docusaurus/core is already on the latest 3.x line (^3.10.0 → resolves to 3.10.1), so there's no upstream release to wait on; the patches are available today and the lockfile just hadn't been refreshed.

Scoped name@range overrides are used for path-to-regexp, picomatch, and ajv so that only the vulnerable ranges are bumped — parallel major versions in the dep tree are left untouched.

Verification

  • npm audit in docs/: found 0 vulnerabilities
  • uv lock --check: passes
  • Pinned versions in regenerated lockfile (spot-checked): postcss 8.5.14, follow-redirects 1.16.0, lodash 4.18.1, path-to-regexp 0.1.13 (vulnerable instance only; 1.9.0 and 3.3.0 left intact), serialize-javascript 7.0.5, picomatch 2.3.2 + 4.0.4, markdown-it 14.1.1, ajv 6.14.0 + 8.18.0, qs 6.15.1, webpack 5.106.2.

Test plan

  • CI green (lint + tests)
  • cd docs && npm install && npm run build succeeds locally
  • Dependabot alerts auto-close after merge to stable

- Add npm overrides in docs/package.json for 12 vulnerable transitive deps
  (postcss, follow-redirects, lodash, path-to-regexp, serialize-javascript,
  node-forge, picomatch, rollup, markdown-it, ajv, qs, webpack) and
  regenerate docs/package-lock.json against current registry.
- Bump requests dev dependency floor to >=2.33.0 to clear
  GHSA-gc5v-m9x4-r6x2 (Insecure Temp File Reuse in extract_zipped_paths).

Resolves the 27 currently open Dependabot alerts. `npm audit` reports
0 vulnerabilities after the lockfile regeneration.
@github-actions github-actions Bot added the type/documentation Improvements or additions to documentation label May 7, 2026
@petercrocker petercrocker marked this pull request as ready for review May 7, 2026 20:15
@petercrocker petercrocker requested a review from a team as a code owner May 7, 2026 20:15
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0cadd41
Status: ✅  Deploy successful!
Preview URL: https://175ab1b7.infrahub-sdk-python.pages.dev
Branch Preview URL: https://shining-freeze.infrahub-sdk-python.pages.dev

View logs

@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff           @@
##           stable     #998   +/-   ##
=======================================
  Coverage   81.41%   81.41%           
=======================================
  Files         134      134           
  Lines       11347    11347           
  Branches     1703     1703           
=======================================
  Hits         9238     9238           
  Misses       1566     1566           
  Partials      543      543           
Flag Coverage Δ
integration-tests 41.84% <ø> (-0.01%) ⬇️
python-3.10 54.34% <ø> (-0.02%) ⬇️
python-3.11 54.36% <ø> (ø)
python-3.12 54.36% <ø> (ø)
python-3.13 54.36% <ø> (ø)
python-3.14 54.34% <ø> (ø)
python-filler-3.12 22.74% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@ajtmccarty ajtmccarty left a comment

Choose a reason for hiding this comment

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

backend bump looks good. don't know about the frontend, requesting their review

@ajtmccarty ajtmccarty requested a review from a team May 7, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants