Skip to content

fix: require Pillow 13 to resolve security advisories - #370

Merged
andremun merged 1 commit into
mainfrom
codex/fix-pillow-security
Oct 24, 2026
Merged

andremun merged 1 commit into
mainfrom
codex/fix-pillow-security

Conversation

@andremun

Copy link
Copy Markdown
Owner

The lockfile contains Pillow 12.3.0, which has known image decoding, memory safety, and Ghostscript vulnerabilities. Require Pillow >=13.0.0 for fresh installations and update the lockfile to 13.0.0. Add the security update to the release notes.

Compatibility: Additive. Pillow is the only package record changed in the lockfile. ISA algorithms, numerical dependency series, public interfaces, and persisted schemas remain unchanged.

Validation on Python 3.12:

  • All 1,137 strict tests pass with warnings treated as errors and 92.84% coverage, including plotting and exports.
  • pip-audit reports 37 vulnerabilities before the update and none afterward across all 80 locked packages, with no skipped packages.
  • Ruff, strict mypy, and Black checks pass.
  • The documentation build checks all 42 pages, 74 option fields, local links, and anchors.
  • Poetry lock validation, dependency consistency, source and wheel builds, and git diff checks pass.
  • Built wheel metadata includes Requires-Dist: pillow (>=13.0.0).

GitHub reports 41 alerts on the default branch at push time. That count is separate from the pip-audit result above. GitHub's alert status needs to be checked after merging.

Advisories include CVE-2026-112398 and CVE-2026-112900.

Require Pillow 13.0.0 or later for fresh installations and update the lockfile from 12.3.0 to 13.0.0. Record the security update in the release notes.

Validation: all 1,137 tests pass with 92.84% coverage. The audit reports no known vulnerabilities across 80 locked packages. Lint, type, format, documentation, and package build checks pass.
@codecov

codecov Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.46%. Comparing base (6abbe2b) to head (9488c46).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #370   +/-   ##
=======================================
  Coverage   91.46%   91.46%           
=======================================
  Files          28       28           
  Lines        5919     5919           
  Branches      736      736           
=======================================
  Hits         5414     5414           
  Misses        325      325           
  Partials      180      180           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andremun
andremun marked this pull request as ready for review October 24, 2026 09:04
Copilot AI balanced review requested due to automatic review settings October 24, 2026 09:04
@andremun
andremun merged commit 0026a0a into main Oct 24, 2026
6 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

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.

2 participants