Skip to content

chore: address cargo audit advisories#160

Merged
rubenhensen merged 1 commit into
mainfrom
chore/cargo-audit-159
May 17, 2026
Merged

chore: address cargo audit advisories#160
rubenhensen merged 1 commit into
mainfrom
chore/cargo-audit-159

Conversation

@dobby-coder
Copy link
Copy Markdown
Contributor

@dobby-coder dobby-coder Bot commented May 16, 2026

Summary

Addresses the advisories raised in #159.

Crate Advisory Status
lettre 0.11.21 → 0.11.22 RUSTSEC-2026-0141 (critical, CVSS 9.1) Fixed in this PR
rustls-pemfile 1.0.4 RUSTSEC-2025-0134 (unmaintained, warning) Out of scope — see below

lettre bump

cargo update -p lettre brings the patched release. The advisory only affects the Boring TLS backend; cryptify uses the default backend, so impact is precautionary. Cargo.toml constraint (lettre = \"0.11.19\") already allows the new version — lockfile-only change.

rustls-pemfile (not fixable here)

The unmaintained rustls-pemfile 1.x is pulled transitively:

rustls-pemfile 1.0.4
└── reqwest 0.11.27
    └── irma 0.2.1
        ├── pg-core 0.6.0 → cryptify
        └── cryptify (direct: irma = \"0.2.1\")

cryptify cannot eliminate this without an upstream bump in irma (currently pinned to reqwest 0.11). Tracking is appropriate at the postguard / irma level, not here. After this PR merges, #159 can be closed — the actionable item for cryptify is done.

Verification

  • cargo audit — vulnerability gone, only the upstream rustls-pemfile warning remains
  • cargo build --release — clean
  • cargo test — 87 passed, 0 failed
  • cargo fmt --all -- --check — clean
  • cargo clippy --all-targets — clean

Reviewer quickstart

```
git fetch origin && git checkout chore/cargo-audit-159 && cargo build --release && cargo test
```

Closes #159

Addresses critical advisory (CVSS 9.1): TLS hostname verification
was disabled when using the Boring TLS backend. cryptify uses the
default backend; bump is precautionary and aligns with the advisory
fix range.

Closes #159
@dobby-coder dobby-coder Bot requested a review from rubenhensen May 16, 2026 22:30
Copy link
Copy Markdown
Contributor Author

@dobby-coder dobby-coder Bot left a comment

Choose a reason for hiding this comment

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

Self-authored PR so cannot formally approve, but verified locally and the change is good:

  • cargo audit — RUSTSEC-2026-0141 (lettre) gone; only the unfixable transitive rustls-pemfile warning via irma → reqwest remains, matching the PR description.
  • Diff is Cargo.lock-only. lettre bumped 0.11.21 → 0.11.22; other transitive deltas are incidental from the update resolution.
  • CI green on all required checks.

Ready to merge from this agent's side.

@rubenhensen rubenhensen merged commit 6a003d2 into main May 17, 2026
6 checks passed
@rubenhensen rubenhensen deleted the chore/cargo-audit-159 branch May 17, 2026 16:06
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.

chore: update dependencies

1 participant