Repository navigation
Drop vulnerable pillow 12.3.0 from the kernel lock - #854
Merged
Merged
Conversation
Pillow's fixes only exist in 13.0.0, which needs Python 3.11, so the lock kept 12.3.0 for Python 3.10 and Dependabot's alert stayed open. Restrict the test extra's matplotlib (pillow's only dependent) to Python 3.11+; the kernel images run 3.12 without that extra. Bump the kernel to 0.6.4 since 0.6.3 is already published. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Kzn6V5vUWgWkgjcEVvTUDu
✅ Deploy Preview for flowfile-wasm canceled.
|
|
Claude finished @Edwardvaneechoud's task in 19s —— View job Review of PR #854Verdict: looks good, with one minor note. The change does what it says:
Minor note
I ran no tests myself. I read the diff and checked how matplotlib is used in |
Edwardvaneechoud
added a commit
that referenced
this pull request
Oct 25, 2026
The dependency fixes on main (#853, #854, #855) conflicted only on poetry.lock's content-hash. The lock was relocked with Poetry 2.4.3, the version that wrote it, which keeps every locked version: besides the new hash it adds the groups line polars-runtime-64 was missing, and poetry check --lock passes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Closes the remaining Dependabot alert on
pillowinkernel_runtime/poetry.lock.The fixes exist only in Pillow 13.0.0, which requires Python 3.11. Because
kernel_runtimesupports Python 3.10, the lock kept a secondpillow 12.3.0entry for 3.10. Dependabot flags that entry, so the alert stayed open after #853.Changes
kernel_runtime/pyproject.toml: limit thetestextra'smatplotlib(Pillow's only dependent) topython >= 3.11. The published kernel images run Python 3.12 and don't install thetestextra, so the images don't change.kernel_runtime/poetry.lock: re-locked. The only change is thatpillow 12.3.0is dropped and13.0.0stays.kernel_runtimeversion and__version__, theimages.pypins, and the regeneratedkernel_image_manifest.json. 0.6.3 is already on Docker Hub, so a lock change without a bump fails the kernel release guard.Testing
pip-auditon the new kernel lock: no known vulnerabilities.tools/check_kernel_version_sync.py: pins in sync at 0.6.4.poetry check --lockpasses forkernel_runtime.pytest tools/tests(18 passed) andpytest flowfile_core/tests/kernel -m "not kernel and not docker_integration"(26 passed).Generated by Claude Code