Skip to content

fix: resolve cargo-deny advisory failures - #41

Merged
paolodamico merged 1 commit into
mainfrom
fix/cargo-deny-advisories
Aug 24, 2026
Merged

fix: resolve cargo-deny advisory failures#41
paolodamico merged 1 commit into
mainfrom
fix/cargo-deny-advisories

Conversation

@thomas-waite

@thomas-waite thomas-waite commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

The Cargo deny / advisories CI job is failing on PRs (e.g. #39).

What & why:

  • Bump ruint 1.18.0 → 1.20.0 — fixes RUSTSEC-2026-0220 (incorrect overflow flags on Uint shifts, potential DoS via formatting).
  • Bump h2, anyhow, crossbeam-epoch — fixes RUSTSEC-2026-0258, -0190, -0204, which the advisory DB now flags (CI will pick these up on its next DB refresh).
  • Bump spin to non-yanked patch releases — clears yanked-crate warnings.
  • Ignore RUSTSEC-2026-0247/0248/0251 (bitmaps/im/sized-chunks unmaintained) in deny.toml — pulled in by provekit's Noir compiler fork under walletkit-core; no upgrade path from this repo until upstream migrates to imbl.

All lockfile-only, semver-compatible bumps. Verified locally: cargo deny check advisories, cargo deny check bans licenses sources, cargo check --workspace --all-targets --all-features, and cargo test --all-features all pass.

Bump ruint, h2, anyhow, crossbeam-epoch, and spin in Cargo.lock; ignore
unmaintained im/sized-chunks/bitmaps advisories from provekit's Noir tree.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@thomas-waite

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: b280595f0e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@paolodamico
paolodamico merged commit 1ec70f7 into main Aug 24, 2026
11 checks passed
@paolodamico
paolodamico deleted the fix/cargo-deny-advisories branch August 24, 2026 21:41
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