Repository navigation
fix(deps): eliminate vulnerable zip 6 and 7 resolutions - #1441
Merged
Merged
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
Change
Cargo Audit reports RUSTSEC-2026-0367, RUSTSEC-2026-0368, and RUSTSEC-2026-0369 for zip 6.0.0 and 7.2.0. All five affected lockfiles now resolve zip 9.0.5. The runtime lockfile contains no zip.
Upgrade direct archive dependencies and propagate zip 9 filename decoding failures. Keep Burn components at 0.20.1 and import the existing CPU components directly. This removes the unused Candle dependency from lock resolution. The
burn-corealias retains the name required by Burn derive macros.Prediction now uses the existing native DuckDB patch at 1.10504.0. This replaces upstream 1.10501.0, whose downloader required vulnerable zip. Arrow remains at its existing 58.3.0 resolution. The optional downloader also uses zip 9.0.5.
Issue relationship
None
Validation
git diff --check: passed.Official advisories: 0367, 0368, 0369.
Runtime impact and rollback
Merged as
25f603cfb1e423567e3f2a8a80fc0491b4faf00dunder explicit user authorization on 2026-10-19 at 14:06:52 UTC. All three automatic main checks passed. Deployment, cloud resource creation, and workflow re-enablement remain excluded. Reverting the fix restores the previous dependency graph, including vulnerable zip versions.Post-merge verification
Merged main
25f603cfb1e423567e3f2a8a80fc0491b4faf00dpassed Monorepo CI, Prediction Markets CI, and Security. The actual main audit artifact reports six workspaces and zero vulnerabilities.The existing Release/GHCR automation succeeded using the verified Security CI image without rebuilding. Immutable digest readback passed. Both Alibaba-calling workflows remain disabled. No deployment, cloud resource creation, workflow re-enablement, or other PR merge was performed.