Skip to content

fix(tls): avoid unverified OCSP status claims - #947

Merged
ryanfowler merged 1 commit into
mainfrom
fix/tls-ocsp-staple-status
Aug 7, 2026
Merged

fix(tls): avoid unverified OCSP status claims#947
ryanfowler merged 1 commit into
mainfrom
fix/tls-ocsp-staple-status

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • require an OCSP CertID match before showing an embedded certificate status
  • show a neutral staple-present indication when the issuer is unavailable or no response matches
  • remove semantic status colors from OCSP data that is not signature and freshness validated
  • document the conditional OCSP inspection output

Tests

  • cargo test --locked --all-features --lib tls::inspect::tests::parse_ocsp_status -- --nocapture
  • cargo test --locked --all-features --lib tls::inspect::tests::render_ocsp_status -- --nocapture
  • cargo fmt --check
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --locked --all-features --lib --bins
  • cargo test --locked --all-features --test cli --test formatting --test grpc --test har --test http --test install --test network --test terminal --test update --test websocket -- --test-threads=2

@ryanfowler
ryanfowler enabled auto-merge August 7, 2026 14:37
@ryanfowler
ryanfowler merged commit cbebc51 into main Aug 7, 2026
7 of 8 checks passed
@ryanfowler
ryanfowler deleted the fix/tls-ocsp-staple-status branch August 7, 2026 14:46
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.

1 participant