Skip to content

Update rustls to fix a security advisory - #2733

Merged
fnando merged 1 commit into
mainfrom
bump-rustls-advisory
Sep 17, 2026
Merged

fnando merged 1 commit into
mainfrom
bump-rustls-advisory

Conversation

@fnando

@fnando fnando commented Sep 17, 2026

Copy link
Copy Markdown
Member

What

Bumps rustls 0.23.310.23.45 in Cargo.lock and drops a stale advisory ignore from deny.toml.

Why

cargo deny check advisories was failing on RUSTSEC-2026-0285 (rustls accepting TLS 1.3 handshake messages at the wrong encryption level), fixed in >=0.23.45. The refresh also bumped rand to 0.8.7, resolving RUSTSEC-2026-0097, so its now-unused ignore entry was removed.

Known limitations

N/A

Copilot AI balanced review requested due to automatic review settings September 17, 2026 19:03
@github-project-automation github-project-automation Bot moved this to Backlog (Not Ready) in DevX Sep 17, 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.

🟢 Approval recommended

No unresolved review issues remain.

Pull request overview

Updates Rust TLS dependencies to address a security advisory and removes an obsolete advisory ignore.

Changes:

  • Upgrades rustls and rustls-webpki in Cargo.lock.
  • Removes the resolved rand advisory ignore from deny.toml.
File summaries
File Summary
deny.toml Removes the obsolete advisory ignore.
Cargo.lock Updates patched TLS dependency versions and checksums.
Review details
  • Files reviewed: 1/2 changed files
  • Comments generated: 0
  • Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@fnando
fnando enabled auto-merge (squash) September 17, 2026 19:49
@fnando fnando self-assigned this Sep 17, 2026
@fnando fnando moved this from Backlog (Not Ready) to Needs Review in DevX Sep 17, 2026
@fnando
fnando requested review from a team and leighmcculloch September 17, 2026 19:50
@fnando
fnando merged commit 6376025 into main Sep 17, 2026
236 checks passed
@fnando
fnando deleted the bump-rustls-advisory branch September 17, 2026 22:58
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in DevX Sep 17, 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