Skip to content

fix: update rustls to 0.23.46 to resolve new RUSTSEC advisories - #855

Merged
bfjelds (bfjelds) merged 1 commit into
mainfrom
user/bfjelds/rustls-0348-fix
Oct 14, 2026
Merged

bfjelds (bfjelds) merged 1 commit into
mainfrom
user/bfjelds/rustls-0348-fix

Conversation

@bfjelds

Copy link
Copy Markdown
Member

Description

Update rustls from 0.23.45 to 0.23.46 in both the root Cargo.lock and tools/tailor/Cargo.lock. Regenerate NOTICE for the root workspace. No source or manifest changes.

Rationale

#847 bumped rustls to 0.23.45 to fix RUSTSEC-2026-0285. Since then, three new advisories were published against 0.23.45 itself:

  • RUSTSEC-2026-0347: client can leak plaintext application data to the wrong server after an ECH rejection.
  • RUSTSEC-2026-0348: Brotli certificate decompression can allocate up to 1 GiB before authentication.
  • RUSTSEC-2026-0349: unauthenticated ServerHello can downgrade later connections to classical (non-PQ) key exchange.

All three are fixed in rustls >=0.23.46. This is currently failing cargo-deny on main and blocking PR #842 (tools/tailor resolves rustls 0.23.45 via oci-client -> reqwest/hyper-rustls/tokio-rustls).

Validation

Source Command or check Result Notes
Agent cargo update -p rustls (root workspace) Passed rustls 0.23.45 -> 0.23.46; 257 other deps unchanged.
Agent cargo update -p rustls (tools/tailor workspace) Passed rustls 0.23.45 -> 0.23.46; 116 other deps unchanged.
Agent cargo deny check in tools/tailor Passed advisories ok, bans ok, licenses ok, sources ok - this is the lockfile the cargo-deny CI job actually checks.
Agent cargo deny check at repo root Pre-existing failures unrelated to rustls No rustls advisory remains; root-level cargo-deny is not wired into CI (only tools/tailor is, via .github/workflows/tailor.yml), and the remaining failures (banned ring, several unmaintained-crate warnings, h2/quick-xml advisories) predate this change.
Agent cargo check --locked --workspace in tools/tailor Passed
Agent cargo check --locked -p trident -p trident-acl-agent --features dangerous-options,grpc-preview Passed
Agent cargo test --locked -p trident-acl-agent --lib Passed 193 tests passed.
Agent make update-notice Passed Regenerated NOTICE; only the rustls version entry changed.
Agent git diff --check Passed No whitespace errors.
CI PR validation Pending Not yet observed for this commit.

RUSTSEC-2026-0347/0348/0349 were published against rustls 0.23.45
(the version #847 bumped to for RUSTSEC-2026-0285), making it
vulnerable again: an ECH-rejection plaintext leak, an unbounded
brotli decompression allocation, and a PQ key-exchange downgrade.
All are fixed in rustls 0.23.46.

Updates both the root Cargo.lock (ACL agent kube client path) and
tools/tailor/Cargo.lock (oci-client/reqwest path, the lockfile
cargo-deny CI actually checks). Regenerates NOTICE for the root
workspace. No source or manifest changes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

@bfjelds
bfjelds (bfjelds) marked this pull request as ready for review October 14, 2026 21:33
@bfjelds
bfjelds (bfjelds) requested a review from a team as a code owner October 14, 2026 21:33
Copilot AI balanced review requested due to automatic review settings October 14, 2026 21:33

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

The narrowly scoped dependency updates are consistent, reported validation supports them, and no blocking issues remain.

0 open findings

What changed in this PR

Updates rustls to 0.23.46 in both workspaces to address three security advisories and unblock Tailor dependency checks.

Changes:

  • Updates rustls versions and checksums in both lockfiles.
  • Refreshes the root dependency notice; no source or manifest changes.
File Description
tools/​tailor/​Cargo.lock Updates Tailor’s rustls dependency.
NOTICE Records the updated rustls version.
Cargo.lock Updates the root workspace’s rustls dependency.

🧠 Review effort: Balanced


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

@bfjelds
bfjelds (bfjelds) merged commit 0b76b52 into main Oct 14, 2026
104 checks passed
@bfjelds
bfjelds (bfjelds) deleted the user/bfjelds/rustls-0348-fix branch October 23, 2026 04:24
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.

3 participants