Skip to content

Update rustls to fix cargo audit - #188

Merged
lorisleiva merged 1 commit into
mainfrom
loris/fix-cargo-audit
Oct 16, 2026
Merged

lorisleiva merged 1 commit into
mainfrom
loris/fix-cargo-audit

Conversation

@lorisleiva

Copy link
Copy Markdown
Member

This PR fixes the red Audit job on main by bumping rustls to 0.23.46 in the lockfile, which resolves four new RustSec vulnerabilities (RUSTSEC-2026-0346 to 0349) in transitive dependencies of the solana-client stack.

It also removes the one stale --ignore flags (RUSTSEC-2024-0376) from make audit, since none of those advisories fire any more. Keeping them would not save us future work. Because Cargo.lock is checked in, an advisory only comes back if we reintroduce a vulnerable version, for example through a downgrade or a new dependency pulling in an older major line. When that happens we want CI to flag it, rather than have a stale ignore quietly let it through. If one does come back and can't be bumped, we can re-add the ignore with a comment explaining why. The remaining ignores still fire on old major lines with no compatible patch.

This PR fixes the red Audit job on main by bumping `rustls` to 0.23.46 in the lockfile, which resolves four new RustSec vulnerabilities (RUSTSEC-2026-0346 to 0349) in transitive dependencies of the `solana-client` stack.

It also removes the one stale `--ignore` flags (RUSTSEC-2024-0376) from `make audit`, since none of those advisories fire any more. Keeping them would not save us future work. Because `Cargo.lock` is checked in, an advisory only comes back if we reintroduce a vulnerable version, for example through a downgrade or a new dependency pulling in an older major line. When that happens we want CI to flag it, rather than have a stale ignore quietly let it through. If one does come back and can't be bumped, we can re-add the ignore with a comment explaining why. The remaining ignores still fire on old major lines with no compatible patch.
@lorisleiva
lorisleiva merged commit 3e424bd into main Oct 16, 2026
30 checks passed
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.

1 participant