Skip to content

Patch critical and high Dependabot alerts - #461

Merged
dylanuys merged 7 commits into
testnetfrom
fix/high-security-deps
Sep 19, 2026
Merged

dylanuys merged 7 commits into
testnetfrom
fix/high-security-deps

Conversation

@dylanuys

@dylanuys dylanuys commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pin the tested security updates: GitPython 3.1.62, Pillow 12.3.0, aiohttp 3.14.3, cryptography 50.0.1, FastAPI 0.141.1, and Starlette 1.6.0.
  • Regenerate uv.lock so requirement metadata matches pyproject.toml, including GitPython. Exact pins prevent unlocked installations selecting untested newer versions of these packages.
  • Leave torch 2.7.1 and transformers 4.57.6 unchanged; their outstanding advisories require a separate GPU/model compatibility upgrade.

Also brings main's existing uniform-random challenge allocation and 5.0.7 version bump into testnet.

Validation

  • 204 tests passed with the pinned dependency versions; CI-selected Ruff checks and git diff --check passed.
  • uv lock --check --offline passed.
  • Local callback server started; unsigned requests rejected, signed uploads accepted/staged, and retries did not enqueue duplicate work. No production requests or burn transactions were sent.

dylanuys and others added 6 commits September 15, 2026 18:48
Promote generator qualification and reward fixes to main
Promote unanswered-challenge skip to main
Bucketed qualified/onboarding/probe slots were starving some miners; the default is a uniform draw again, with buckets still available as an opt-in.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Restore uniform random generator challenge sampling
Pin GitPython, Pillow, aiohttp, cryptography, FastAPI, and Starlette to patched releases so the GitPython RCE and other high-severity lockfile advisories clear without bumping torch or transformers.

Co-authored-by: Cursor <cursoragent@cursor.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

uv.lock metadata is inconsistent with pyproject.toml for the GitPython constraint, indicating the lockfile may not be fully regenerated from the updated project requirements.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR updates Python dependency constraints and the uv lockfile to address critical/high Dependabot alerts by moving to patched versions of key packages (notably GitPython, cryptography, aiohttp, Pillow, FastAPI/Starlette).

Changes:

  • Updated pyproject.toml dependency constraints to require patched versions for vulnerable packages.
  • Refreshed uv.lock to resolve to updated package versions (e.g., aiohttp 3.14.3, cryptography 50.0.1, GitPython 3.1.62, Pillow 12.3.0, FastAPI 0.141.1, Starlette 1.6.0).
File summaries
File Description
pyproject.toml Updates dependency constraints to require patched versions addressing security advisories.
uv.lock Re-locks the environment to the newer, patched dependency versions and their artifacts.
Review details
  • Files reviewed: 1/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pyproject.toml Outdated
Comment on lines +26 to +29
"GitPython>=3.1.62", # 3.1.62 closes CRITICAL GHSA-284h-m62q-gf8w and later HIGH git option/RCE advisories
"cryptography>=50.0.0", # 50.0.0 closes HIGH PKCS#7 Bleichenbacher oracle
"fastapi>=0.115.14",
"starlette>=1.3.1", # 1.3.1 closes HIGH multipart/form DoS and StaticFiles SSRF
@dylanuys
dylanuys changed the base branch from main to testnet September 19, 2026 16:30
@dylanuys
dylanuys merged commit ed7d5bb into testnet Sep 19, 2026
2 checks passed
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.

3 participants