Skip to content

⬆️(dependencies) pin pillow to v12.2.0 [SECURITY]#668

Merged
NathanVss merged 1 commit intomainfrom
fix/pillow
Apr 15, 2026
Merged

⬆️(dependencies) pin pillow to v12.2.0 [SECURITY]#668
NathanVss merged 1 commit intomainfrom
fix/pillow

Conversation

@NathanVss
Copy link
Copy Markdown
Contributor

Summary

  • Pillow 12.1.1 is flagged for CVE-2026-40192 — a FITS GZIP
    decompression bomb that can be triggered by a crafted image.
  • We don't depend on pillow directly: it comes in through
    easy_thumbnails, which has no upper bound, so a plain version bump
    isn't an option.
  • Added a [tool.uv] constraint-dependencies entry pinning
    pillow>=12.2.0 so any transitive pull picks up the patched release.

Pillow 12.1.1 is vulnerable to CVE-2026-40192 (FITS GZIP decompression
bomb). easy_thumbnails pulls pillow transitively with no upper bound,
so enforce the floor via a uv constraint rather than a direct
dependency bump.
@sonarqubecloud
Copy link
Copy Markdown

@NathanVss NathanVss requested a review from qbey April 14, 2026 14:15
Comment on lines +96 to +100
constraint-dependencies = [
# Pin pillow above the version vulnerable to CVE-2026-40192
# (FITS GZIP decompression bomb). Pulled transitively via
# easy_thumbnails, which has no upper bound on pillow.
"pillow>=12.2.0",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure we need this, since this is the purpose of the lock file.

Copy link
Copy Markdown
Contributor Author

@NathanVss NathanVss Apr 15, 2026

Choose a reason for hiding this comment

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

I was concerned by the fact that the next uv lock|add|remove would override the value in the lock file automatically 🤔 Do you think we are safe or not doing this only in the lock file?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think uv would downgrade a dependency except if a lib is requiring it explicitly. I don't have a strong opinion on this, you are right in the fact that it's explicit and if some dependency tries to downgrade, it will raise directly, instead of being detected afterward. So it's a good idea ;)

@NathanVss NathanVss merged commit 832725d into main Apr 15, 2026
30 of 31 checks passed
@NathanVss NathanVss deleted the fix/pillow branch April 15, 2026 08:38
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