fix(deps): bump click/cryptography/pip floors + clerk 6→7 (pip-audit --strict, CI #590) - #214
Draft
Sbussiso wants to merge 1 commit into
Draft
Conversation
… --strict, CI #590) CI run #590 (32779720467) failed at the "Dependency scan (pip-audit)" step on the dependabot eslint branch: pip-audit --strict reported 5 known vulnerabilities in 3 transitive packages after new 2026-08 advisories landed in the PyPA Advisory DB: click 8.3.1 PYSEC-2026-2132 -> 8.3.3 cryptography 48.0.1 PYSEC-2026-3552 -> 50.0.0 cryptography 48.0.1 PYSEC-2026-3553 -> 49.0.0 cryptography 48.0.1 PYSEC-2026-3554 -> 49.0.0 pip 26.1.2 PYSEC-2026-3721 -> 26.2 Root cause: the constraint-dependencies floors in backend/pyproject.toml were at the old fixed versions (pip>=26.1.2, cryptography>=48.0.1, no click constraint), and uv.lock pinned those exact versions. The three new advisories are not satisfied. Fix: raise the floors (pip>=26.2, cryptography>=50.0.0, add click>=8.3.3) and regenerate uv.lock. cryptography>=50.0.0 is unreachable while clerk-backend-api 6.0.1 is resolved (it caps cryptography <49.0.0), so the resolver upgrades clerk-backend-api 6.0.1 -> 7.0.0, which was released 2026-08-11 specifically to raise the cap to <51.0.0. 50.0.0 satisfies all three cryptography advisories. clerk-backend-api 6->7 is a major bump with documented breaking API shape changes, but the app only uses Clerk (client) and AuthenticateRequestOptions (security); the full 699-test suite passes locally with the bump applied (in-memory DB, mocked auth). Verified locally (mirrors the CI job): - uv run ruff check -> All checks passed - uv run pip-audit --strict -> No known vulnerabilities found - uv run pytest -q -> 699 passed Refs: CI run https://github.com/SourceBox-LLC/Sentinel-Command/actions/runs/32779720467 Dependabot PR #201 (eslint bump) this branch carries
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.
Why
CI run #590 (run 32779720467) — the
Test & Deploycheck on Dependabot PR #201 (eslint 9.39.4 → 10.9.0) — failed at the Dependency scan (pip-audit) step.uv run pip-audit --strictreported 5 known vulnerabilities in 3 packages after new 2026-08 advisories landed in the PyPA Advisory DB:The
Backend testsjob runs on every PR to master (including frontend-only Dependabot bumps), so a stale backend lockfile blocks this eslint PR even though the eslint change is unrelated.Root cause
The
constraint-dependenciesfloors inbackend/pyproject.tomlwere still at the old fixed versions (pip>=26.1.2,cryptography>=48.0.1, noclickconstraint), anduv.lockpinned those exact versions. The three new 2026-08 advisories are not satisfied by the old floors.Fix
Raise the floors —
pip>=26.2,cryptography>=50.0.0, addclick>=8.3.3— and regenerateuv.lock:click8.3.1 → 8.4.2 (fixes PYSEC-2026-2132)pip26.1.2 → 26.2.1 (fixes PYSEC-2026-3721)cryptography48.0.1 → 50.0.0 (fixes all three: 3552/3553/3554)clerk-backend-api6.0.1 → 7.0.0 (forced: clerk 6.0.1 capscryptography<49.0.0, incompatible with>=50.0.0; clerk 7.0.0 was released 2026-08-11 specifically to raise the cap to<51.0.0)The 7.0.0 release notes list several Breaking API-shape changes (error/verification types on
email_addresses,phone_numbers,sessions,role_sets,instance_settings, etc.). The app only usesclerk_backend_api.Clerk(the client) andclerk_backend_api.security.AuthenticateRequestOptions— neither is in the breaking-changes list — and the full 699-test suite passes with the bump applied (in-memory DB, mocked auth). Still flagging for review attention since the breaking changes are real and a live-credentials smoke test is worth doing before deploy.Verified locally (mirrors the CI job)
Note
This is the same root cause addressed by sibling draft PRs #207 and #208 (created by parallel CI triage runs minutes apart). Opening this one anyway per the per-run triage contract; recommend the human pick one and close the rest to avoid duplicate review load. PR #185 (the Dependabot clerk 6→7 bump) is the upstream fix for the cryptography cap but doesn't itself touch the click/pip constraints, so it's complementary rather than a substitute.