Skip to content

ci: clear pip-audit advisory backlog blocking eslint PR #201 (run #32779906420) - #213

Draft
Sbussiso wants to merge 1 commit into
dependabot/npm_and_yarn/frontend/eslint-10.9.0from
ci-fix/32779906420
Draft

ci: clear pip-audit advisory backlog blocking eslint PR #201 (run #32779906420)#213
Sbussiso wants to merge 1 commit into
dependabot/npm_and_yarn/frontend/eslint-10.9.0from
ci-fix/32779906420

Conversation

@Sbussiso

Copy link
Copy Markdown
Contributor

CI triage — unblock eslint PR #201 (run #32779906420)

The Dependabot eslint 9.39.4 → 10.9.0 bump (PR #201) went red in the Backend tests job, at the "Dependency scan (pip-audit)" step — not because of the eslint change. pip-audit --strict flagged 5 newly-landed advisories in 3 backend packages:

Package Version Advisory Fix Path
click 8.3.1 PYSEC-2026-2132 8.3.3 transitive via uvicorn
cryptography 48.0.1 PYSEC-2026-3552 50.0.0 transitive via clerk-backend-api / authlib
cryptography 48.0.1 PYSEC-2026-3553 49.0.0 (as above)
cryptography 48.0.1 PYSEC-2026-3554 49.0.0 (as above)
pip 26.1.2 PYSEC-2026-3721 26.2 dev/CI surface only

The frontend bump itself is fine — the Frontend audit + build job passed.

Fix

Follows the repo's existing constraint-dependencies pattern in backend/pyproject.toml (same approach already used for authlib, urllib3, idna, starlette, joserfc, msgpack, pydantic-settings): raise the transitive floor so uv resolves past the advisory, with a comment citing the PYSEC ID.

  • pip >=26.1.2>=26.2
  • cryptography >=48.0.1>=50.0.0
  • click — new entry >=8.3.3

uv lock resolves cleanly. It also pulls clerk-backend-api 6.0.1 → 7.0.0, because the 6.x line caps cryptography < 49, which blocked the 50.0.0 floor. The backend uses only stable clerk API surface (Clerk(), clerk.organizations.*, clerk.organization_memberships.list, authenticate_request, AuthenticateRequestOptions) — all still exported in 7.0.0.

Verified locally (mirrors the CI job)

  • uv run ruff checkAll checks passed!
  • uv run pip-audit --strictNo known vulnerabilities found ← the step that failed
  • uv run pytest -q699 passed

Notes

  • These are transitive floors, not direct deps — each line is meant to be removed once the carrying direct dep's own pin clears the version (per the inline comments).
  • Drafted against the dependabot branch so merging this lands the backend audit fix together with the eslint bump.

Hermes coder automated CI triage

Run #32779906420 (Dependabot eslint 9.39.4 -> 10.9.0) failed the
Backend tests > "Dependency scan (pip-audit)" step on 5 newly-landed
advisories in 3 backend packages — unrelated to the frontend eslint bump:

  click        8.3.1  PYSEC-2026-2132  (fix 8.3.3)   transitive via uvicorn
  cryptography 48.0.1 PYSEC-2026-3552  (fix 50.0.0)  transitive via
                       PYSEC-2026-3553  (fix 49.0.0)  clerk-backend-api / authlib
                       PYSEC-2026-3554  (fix 49.0.0)
  pip          26.1.2 PYSEC-2026-3721  (fix 26.2)    dev/CI surface only

Per the repo's established constraint-dependencies pattern, bump the
transitive floors so uv resolves past the advisories, and cite the PYSEC
IDs in comments (matching the existing entries for authlib/urllib3/idna/
starlette/joserfc/msgpack/pydantic-settings):

  pip          >=26.1.2  -> >=26.2
  cryptography >=48.0.1  -> >=50.0.0
  click        (new)      >=8.3.3

`uv lock` resolves cleanly, forcing clerk-backend-api 6.0.1 -> 7.0.0 (the
6.x line caps cryptography <49, which blocked 50.0.0). Verified locally:
  - ruff check           : clean
  - pip-audit --strict   : No known vulnerabilities found
  - pytest -q            : 699 passed

These are transitive floors only; remove each line once the carrying
direct dep's own pin clears the version (per inline comments).
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