fix(backend): bump click >=8.3.3 for PYSEC-2026-2132 (CI #509 backend half) - #136
Draft
Sbussiso wants to merge 2 commits into
Draft
fix(backend): bump click >=8.3.3 for PYSEC-2026-2132 (CI #509 backend half)#136Sbussiso wants to merge 2 commits into
Sbussiso wants to merge 2 commits into
Conversation
Bumps [redis](https://github.com/redis/redis-py) from 7.4.0 to 8.0.1. - [Release notes](https://github.com/redis/redis-py/releases) - [Changelog](https://github.com/redis/redis-py/blob/master/CHANGES) - [Commits](redis/redis-py@v7.4.0...v8.0.1) --- updated-dependencies: - dependency-name: redis dependency-version: 8.0.1 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
CI run #509 (30306860966) failed the backend pip-audit --strict gate: click 8.3.1 (transitive via uvicorn) flagged by PYSEC-2026-2132, fixed in 8.3.3. Add a constraint-dependencies pin following the existing pattern (authlib, urllib3, starlette, etc.) so uv resolves click to a patched version. Lockfile updated: click 8.3.1 -> 8.4.2. Verified locally: - uv lock resolves cleanly - uv run pip-audit --strict -> No known vulnerabilities found - uv run ruff check -> All checks passed - uv run pytest -q -> 699 passed Note: the same CI run also failed the frontend npm-audit gate on react-router (GHSA-qwww-vcr4-c8h2). That one has NO safe in-range fix — see the PR body for the full analysis. This PR addresses the backend half only.
Sbussiso
referenced
this pull request
Jul 27, 2026
Bumps [eslint](https://github.com/eslint/eslint) from 9.39.4 to 10.8.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.4...v10.8.0) --- updated-dependencies: - dependency-name: eslint dependency-version: 10.8.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/npm_and_yarn/frontend/eslint-10.8.0
branch
4 times, most recently
from
August 3, 2026 21:33
8633fdf to
19594ab
Compare
Sbussiso
referenced
this pull request
Aug 3, 2026
Bumps [eslint](https://github.com/eslint/eslint) from 9.39.4 to 10.8.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.4...v10.8.0) --- updated-dependencies: - dependency-name: eslint dependency-version: 10.8.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Sbussiso
referenced
this pull request
Aug 3, 2026
Bumps [eslint](https://github.com/eslint/eslint) from 9.39.4 to 10.8.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.4...v10.8.0) --- updated-dependencies: - dependency-name: eslint dependency-version: 10.8.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI triage — run #509 (30306860966)
workflow_runfor Test & Deploy ondependabot/npm_and_yarn/frontend/eslint-10.8.0failed with two independent audit-gate failures, both newly-disclosed CVEs hitting an existing branch (neither caused by the eslint bump in this PR):1. Backend —
pip-audit --strict❌ → ✅ fixed in this PRclick 8.3.1flagged by PYSEC-2026-2132 (transitive viauvicorn).8.3.3. Patch bump within 8.3.x — API-compatible."click>=8.3.3"to[tool.uv] constraint-dependenciesinbackend/pyproject.toml, following the existing pattern (authlib, urllib3, idna, starlette, cryptography, joserfc, msgpack, pydantic-settings).uv.lockupdated:click 8.3.1 → 8.4.2.uv lock→ resolved cleanlyuv run pip-audit --strict→No known vulnerabilities found(exit 0)uv run ruff check→All checks passeduv run pytest -q→ 699 passed2. Frontend —
npm audit --audit-level=high --omit=dev❌ → NOT fixed here (no safe fix)react-router 7.18.1flagged by GHSA-qwww-vcr4-c8h2 — "RSC Mode CSRF Bypass Allows Action Execution Before 400 Response" (high, CWE-352). Affectsreact-router >=7.12.0 <8.3.0.react-router@8.3.0, a SemVer major bump. In v8 thereact-router-dompackage was removed (no 8.x release exists) and exports were restructured (BrowserRouter/Routes/Link/useNavigateetc. moved to the./domsubpath). The codebase imports fromreact-router-domin 12 files (App.jsx,main.jsx, 8 page components, 4 shared components) — migrating to v8 requires editing all of them and re-testing the entire SPA navigation surface.react-router-dom@7.11.0) is a downgrade that would reopen 3 already-fixed advisories: build(deps-dev): bump vite from 8.0.7 to 8.0.16 in /frontend #17 (RCE, fix 7.14.2), build(deps): bump python-multipart from 0.0.27 to 0.0.31 in /backend #18 (open redirect, fix 7.14.1), build(deps): bump starlette from 1.2.0 to 1.3.1 in /backend #19 (DoS, fix 7.15.0). Not acceptable.BrowserRouter— it does not use RSC mode / framework routes, so the vulnerable code path is likely not reachable. A short-term option is to document non-reachability and temporarily relax the gate (--audit-level=criticalor--ignore-vuln GHSA-qwww-vcr4-c8h2with a revisit date) until a proper v8 migration is scheduled. That's a policy decision for the owner, so I left it out of this PR.What this PR does
Recommendation for the frontend
Either (a) schedule a
react-routerv7→v8 migration as its own PR (touches 12 files + full nav re-test), or (b) if RSC mode is confirmed unreachable, add--ignore-vuln GHSA-qwww-vcr4-c8h2to thenpm auditstep in.github/workflows/deploy.ymlwith a comment + revisit date, matching the pattern already documented in the workflow comments.Auto-triaged by Hermes coder from the
workflow_runwebhook for run #509.