fix(deps): bump cryptography to 50.0.0 and constrain aiohttp >=3.14.3 in strix CI lock - #735
fix(deps): bump cryptography to 50.0.0 and constrain aiohttp >=3.14.3 in strix CI lock#735seonghobae wants to merge 2 commits into
Conversation
cryptography 49.0.0 in requirements-strix-ci.txt is flagged by CVE-2026-69247 / GHSA-g6cj-pr64-35w5 (PKCS#7 decrypt Bleichenbacher oracle, fixed in 50.0.0), which hard-fails the org-wide pip-audit and dependency-review gates on every PR. Bump the direct pin and regenerate requirements-strix-ci-hashes.txt with the recorded uv command; the only transitive change is pyopenssl 26.3.0 -> 26.4.0. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 48 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Comment |
aiohttp 3.14.1 is transitively pulled in by strix-agent and is vulnerable to three advisories published 2026-08-03: GHSA-cq5v-8q36-5273 (HIGH) out-of-bounds heap read in the C HTTP response parser error path (<= 3.14.2) GHSA-mq44-7p77-q5h7 (MEDIUM) WebSocket client accepts compressed frames without negotiated permessage-deflate (<= 3.14.1) GHSA-mfx4-hv73-q22v (MEDIUM) HTTP request smuggling via WebSocket upgrade (<= 3.14.1) All three are cleared by 3.14.3, which fails the required dependency-review gate (fail-on-severity: moderate) and blocks merges. Adds an explicit floor to requirements-strix-ci.txt -- a floor rather than a pin so Dependabot can keep moving it -- and regenerates the lock with the recorded compile command. Only aiohttp changes; the rest of the resolution is unchanged. Co-Authored-By: Claude <noreply@anthropic.com>
|
Closing as folded into the atomic central baseline PR #731. Exact blob reuse moved both the reviewed |
Pull request was closed
What
Two advisory-driven changes to the same hash-pinned lock, kept in one PR because
they regenerate the same file.
requirements-strix-ci.txt:cryptography==49.0.0→==50.0.0aiohttp>=3.14.3(transitive viastrix-agent, so it needs an explicitconstraint line; a floor rather than a pin so Dependabot can keep moving it)
requirements-strix-ci-hashes.txt: regenerated with the exact command recorded in thefile header (
uv pip compile --generate-hashes --python-version 3.13 --python-platform x86_64-manylinux_2_28 …).Transitive movement is
pyopenssl 26.3.0 → 26.4.0andaiohttp 3.14.1 → 3.14.3;everything else is hash-block churn for those packages.
Why
cryptography — CVE-2026-69247 / GHSA-g6cj-pr64-35w5
(PKCS#7 decrypt Bleichenbacher oracle in cryptography ≥44 <50) hard-fails both org-wide
gates on every PR in this repo:
pip-audit (Python dependency audit)("Found 1 knownvulnerability" in
requirements-strix-ci.txt) anddependency-review.aiohttp — three advisories published 2026-08-03 against the pinned
aiohttp==3.14.1:All three are cleared by 3.14.3. They fail the REQUIRED
dependency-reviewcheck(
fail-on-severity: moderate,.github/workflows/security-scan.yml) — confirmed on#734
(run 30891619577, job 91934771440) and
#733 — and would have failed
this PR too, since the cryptography-only revision left aiohttp at 3.14.1.
Merging this unblocks the queue; affected PRs then need a check rerun / sweep retry.
Verification
uv==0.11.25(the version this repopins in
requirements-opencode-review-ci.txt), not hand-edited, per the hash-pinnedrequirements discipline in
CLAUDE.md.exactly one hunk — the
aiohttpversion + hash block, plus the# vialist gaining-r requirements-strix-ci.txtnow that it is a direct constraint. No other packagemoved. Resolution succeeded (106 packages), so
strix-agent==1.0.4accepts 3.14.3.pip-audit/dependency-reviewon this PR are the authoritative check.Related
dependency-reviewcurrently re-flags pre-existing vulnerabilities on every PRbecause
.github/workflows/sbom-generation.ymlhas nopush:trigger, somainnevergets a dependency snapshot ("The number of snapshots compared for the base SHA (0) and
the head SHA (1) do not match"). That root cause is fixed separately in
#736; this PR is still needed
because these advisories are genuinely present in the lock.
🤖 Generated with Claude Code