Skip to content

docs: clarify BoringSSL cipher list configuration - #555

Open
Daiyuanranfeng wants to merge 1 commit into
cloudflare:masterfrom
Daiyuanranfeng:docs/boringssl-cipher-list
Open

Daiyuanranfeng wants to merge 1 commit into
cloudflare:masterfrom
Daiyuanranfeng:docs/boringssl-cipher-list

Conversation

@Daiyuanranfeng

Copy link
Copy Markdown

Fixes #155

Summary

  • Point both context cipher-list setters to BoringSSL’s cipher-list syntax.
  • Explain how the permissive and strict setters handle unrecognized entries, and clarify that neither setter changes TLS 1.3 cipher suites.

Validation

  • Checked the behavior against the BoringSSL header pinned by this repository and verified the linked documentation anchor.
  • cargo fmt --all -- --check and git diff --check passed. No runtime code changed.

@clanky-the-clanker

Copy link
Copy Markdown

This pull request was not reviewed automatically because its author does not have write access to this repository. A maintainer with write access can start a review by commenting clanky review (or clanker review or @clanky-the-clanker review), optionally followed by instructions (e.g. clanky review focus on the retry logic).

This branch has not been deployed

No deployments
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.

Documentation of SslContextBuilder::set_cipher_list does not match boringSSL's documentation

1 participant