Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch default cryptography provider to aws-lc-rs #264

Open
cpu opened this issue Jun 26, 2024 · 1 comment
Open

Switch default cryptography provider to aws-lc-rs #264

cpu opened this issue Jun 26, 2024 · 1 comment

Comments

@cpu
Copy link
Member

cpu commented Jun 26, 2024

The current default is ring:

default = ["std", "ring"]

This is incongruous with rustls as of 0.24 and since this crate is almost entirely designed to be used with rustls we should keep the two in sync. While easily fixed, it helps avoid situations like rustls/rustls-platform-verifier#102

I propose we harmonize the default when the next occasion for a major release arrives.

@djc
Copy link
Member

djc commented Jun 26, 2024

I think we should probably avoid having a crypto implementation in the default features in the next breaking release (though it might not matter much for this crate).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants