Skip to content

Bump rustls to 0.23.45 for RUSTSEC-2026-0285 - #69

Merged
hkd987 merged 1 commit into
mainfrom
feat/bump-rustls-rustsec-2026-0285-cd92
Sep 21, 2026
Merged

hkd987 merged 1 commit into
mainfrom
feat/bump-rustls-rustsec-2026-0285-cd92

Conversation

@hkd987

@hkd987 hkd987 commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Follow-up for cargo-audit going red on main (and on Dependabot #68 clap bump). Do not land this on #68; that PR did not cause the advisory.

The advisory

RUSTSEC-2026-0285 / TLS 1.3 handshake messages incorrectly accepted across encryption level boundaries. Against rustls 0.23.40 (medium, 5.3). Patched in rustls >= 0.23.45. rustls is not a direct dependency — it sits under reqwest (rustls-tls) and lettre (tokio1-rustls-tls).

Reproduced on current main Cargo.lock with cargo-audit 0.22.1: error: 1 vulnerability found! (rustls 0.23.40).

The fix

Same shape as #37 (h2) and #40 (chacha20): lockfile-only, no Cargo.toml pin, no new audit ignore.

cargo update -p rustls on current main:

  • rustls 0.23.40 → 0.23.45 (latest compatible 0.23.x)
  • rustls-webpki 0.103.13 → 0.103.15 (required companion)

Resolver also unified a few windows-sys edges from 0.52.0 to 0.59.0 (already present in the lockfile). No ignore entry: an upstream patch exists.

Verification

  • On main lockfile: cargo-audit 0.22.1 reports RUSTSEC-2026-0285 against rustls 0.23.40.
  • On this branch: ./scripts/cargo-audit-gate.sh → ✓ cargo-audit-gate passed (3 advisories on the documented ignore list).
  • CI cargo audit on this PR head: success.
  • cargo metadata --locked clean
  • cargo check --locked --all-targets clean
  • cargo clippy --locked --all-targets -- -D warnings clean
  • cargo fmt --all -- --check clean
  • cargo test --all --locked — 2141 passed, 0 failed (6 ignored)

Do not merge this PR or #68 from this work.

Open in Web Open in Cursor 

cargo-audit is red on main against rustls 0.23.40. Lockfile-only
update to latest compatible 0.23.x (0.23.45); rustls-webpki follows
0.103.13 -> 0.103.15. No Cargo.toml pin, no new audit ignores.

Co-authored-by: Lundin Matthews <lundinmatthews@me.com>
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Before we can merge this PR, you need to sign our Contributor License Agreement. Please read the CLA and post the following comment to sign:

I have read the CLA Document and I hereby sign the CLA


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@cursor

cursor Bot commented Sep 21, 2026

Copy link
Copy Markdown

I have read the CLA Document and I hereby sign the CLA

@hkd987
hkd987 marked this pull request as ready for review September 21, 2026 12:03

@hkd987 hkd987 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

LGTM — lockfile bump rustls 0.23.40→0.23.45 (+ rustls-webpki) clears RUSTSEC-2026-0285. cargo-audit green; CLA flake ignored. Merging when Check is green.

@hkd987
hkd987 merged commit 6502b9e into main Sep 21, 2026
3 of 4 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants