Skip to content

chore(deps): drop rustls-pemfile and bump criterion#141

Merged
littledivy merged 1 commit into
mainfrom
orch/divybot-568
Jun 14, 2026
Merged

chore(deps): drop rustls-pemfile and bump criterion#141
littledivy merged 1 commit into
mainfrom
orch/divybot-568

Conversation

@divybot

@divybot divybot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Clears two dev-only advisory hits flagged by the daily OSV/RustSec scan, without touching library deps or production code.

  • rustls-pemfile 2.2 is unmaintained (RUSTSEC-2025-0134). The only user was examples/tls_server.rs. The example now parses PEM via rustls_pki_types::pem::PemObject, which is already pulled in transitively through tokio-rustls 0.26 — no new dependency added, and rustls-pemfile is removed from [dev-dependencies].
  • criterion 0.4 pulls atty 0.2 (RUSTSEC-2024-0375 / GHSA-g98v-hv3f-hcfr). Bumped to criterion = "0.5.1", which replaced atty with is-terminal. The unmask bench's API surface (Criterion, Throughput, black_box, criterion_group!, criterion_main!) is unchanged across 0.4 → 0.5.

atty and rustls-pemfile are gone from both Cargo.lock and fuzz/Cargo.lock.

Test plan

  • cargo build --locked --all-features --all-targets
  • cargo test --locked --all-features (all tests pass; pre-existing mismatched_lifetime_syntaxes warning unrelated)
  • cargo bench --bench unmask --no-run
  • cargo fmt -- --check
  • cd fuzz && cargo check --locked
  • grep -E 'atty|rustls-pemfile' Cargo.lock fuzz/Cargo.lock → no matches

Closes denoland/divybot#568

Removes two dev-only advisory hits surfaced by the daily OSV/RustSec
scan:

- RUSTSEC-2025-0134 (rustls-pemfile 2.x unmaintained). The
  `examples/tls_server.rs` PEM parsing moves to `rustls_pki_types::pem`,
  which is already pulled in transitively via `tokio-rustls 0.26` (no
  new dependency added).
- RUSTSEC-2024-0375 / GHSA-g98v-hv3f-hcfr (`atty` unmaintained, pulled
  in by `criterion 0.4`). Bump `criterion` to 0.5.1, which replaces
  `atty` with `is-terminal`.

Co-Authored-By: Divy Srivastava <me@littledivy.com>
@divybot divybot marked this pull request as ready for review June 14, 2026 13:21
@littledivy littledivy merged commit 9cc13ea into main Jun 14, 2026
6 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.

2 participants