Skip to content

Bump rustls to clear the key-exchange downgrade advisory - #2801

Merged
fnando merged 1 commit into
mainfrom
rustls-advisory
Oct 14, 2026
Merged

fnando merged 1 commit into
mainfrom
rustls-advisory

Conversation

@fnando

@fnando fnando commented Oct 14, 2026

Copy link
Copy Markdown
Member

What

Bump rustls from 0.23.45 to 0.23.46 in Cargo.lock. It is a transitive dependency (via reqwest, jsonrpsee-http-client, stellar-rpc-client, and testcontainers/bollard), so only the lockfile changes.

Why

rustls 0.23.45 is flagged by a RUSTSEC advisory: under session resumption a network attacker can force a classical-only key exchange, silently dropping post-quantum confidentiality ("harvest now, decrypt later"). There is no key compromise or authentication bypass. The fix landed in rustls 0.23.46. The advisory was failing cargo deny check advisories on main, so this clears it repo-wide.

Known limitations

N/A

Copilot AI lite review requested due to automatic review settings October 14, 2026 18:47
@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Oct 14, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@fnando
fnando requested review from a team and leighmcculloch October 14, 2026 18:47
@fnando fnando self-assigned this Oct 14, 2026
@fnando fnando moved this from Backlog (Not Ready) to Needs Review in DevX Oct 14, 2026
@fnando
fnando enabled auto-merge (squash) October 14, 2026 18:47
@fnando
fnando merged commit d95abbc into main Oct 14, 2026
150 of 151 checks passed
@fnando
fnando deleted the rustls-advisory branch October 14, 2026 21:27
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in DevX Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants