Skip to content

Centralize TLS parsing for inspection#622

Merged
ryanfowler merged 1 commit into
mainfrom
centralize-tls-parsing
May 27, 2026
Merged

Centralize TLS parsing for inspection#622
ryanfowler merged 1 commit into
mainfrom
centralize-tls-parsing

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Reuse the shared Rustls protocol-version helper in TLS inspection instead of duplicating min/max bound logic.
  • Reuse the shared PEM and client-auth parsing helpers from src/tls/mod.rs for inspection CA material and client certificates.
  • Keep inspection-specific custom verification in place so certificate display and OCSP capture behavior stay unchanged.
  • Add a repository note in AGENTS.md so the shared TLS parsing boundary stays explicit.

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features tls::
  • cargo test --all-features --test integration tls -- --test-threads=1

@ryanfowler ryanfowler enabled auto-merge May 27, 2026 13:57
@ryanfowler ryanfowler merged commit b3eac18 into main May 27, 2026
4 checks passed
@ryanfowler ryanfowler deleted the centralize-tls-parsing branch May 27, 2026 13:57
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