Skip to content

Add support for rustls-platform-verifier#4045

Open
manifest wants to merge 1 commit into
transact-rs:mainfrom
manifest:rustls-platform-verifier
Open

Add support for rustls-platform-verifier#4045
manifest wants to merge 1 commit into
transact-rs:mainfrom
manifest:rustls-platform-verifier

Conversation

@manifest
Copy link
Copy Markdown
Contributor

Does your PR solve an issue?

fixes #4044

Is this a breaking change?

No.

This PR

  • Adds optional dependency on rustls-platform-verifier.
  • Adds new feature flags tls-rustls-aws-lc-rs-platform-verifier and tls-rustls-ring-platform-verifier to enable platform's native CA certificates provided by rustls-platform-verifier.
  • Removes dependency on rustls-native-certs (since rustls-platform-verifier is now recommended over it instead).
  • Makes tls-rustls-ring-native-roots feature flag an alias to tls-rustls-ring-platform-verifier for backwards compatibility reasons.

@manifest manifest force-pushed the rustls-platform-verifier branch 4 times, most recently from d3608f7 to d269d84 Compare September 27, 2025 11:27
@manifest
Copy link
Copy Markdown
Contributor Author

Should feature flag tls-rustls-aws-lc-rs be renamed into tls-rustls-aws-lc-rs-webpki to match tls-rustls-ring-webpki?

@curtisleefulton
Copy link
Copy Markdown

Should feature flag tls-rustls-aws-lc-rs be renamed into tls-rustls-aws-lc-rs-webpki to match tls-rustls-ring-webpki?

I vote yes

@curtisleefulton
Copy link
Copy Markdown

I'm suggesting a new feature on top of this PR. See manifest#2

Feature tls-rustls-no-provider-platform-verifier installs no crypto library and uses CryptoProvider::get_default() instead. This will work the same as Reqwest's rustls-tls-native-roots-no-provider feature, when coupled with rustls-platform-verifier crate .

See https://docs.rs/rustls/latest/rustls/crypto/struct.CryptoProvider.html#using-the-per-process-default-cryptoprovider :

libraries should use ClientConfig::builder()/ServerConfig::builder() or otherwise rely on the CryptoProvider::get_default() provider.

@manifest manifest force-pushed the rustls-platform-verifier branch from 10f376e to 8b4f6b5 Compare May 28, 2026 18:03
@manifest
Copy link
Copy Markdown
Contributor Author

I've updated this PR to use the latest sqlx version.

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.

Add support for rustls-platform-verifier

2 participants