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

cryptography: add std feature #317

Merged
merged 1 commit into from
Sep 30, 2020
Merged

cryptography: add std feature #317

merged 1 commit into from
Sep 30, 2020

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Sep 28, 2020

Adds an off-by-default std feature which activates the std features of all of the other dependencies.

This is a somewhat lousy way of doing this, and I wish that there were a better approach, so much that I posted a Pre-Pre-RFC about it:

https://internals.rust-lang.org/t/pre-pre-rfc-weak-cargo-feature-activation/13141

Edit: seems there's an open issue about this already:

rust-lang/cargo#3494

For now though, this is immediately useful to me.

@tarcieri tarcieri requested a review from newpavlov September 28, 2020 22:44
@tarcieri tarcieri force-pushed the cryptography/std-feature branch from 23da3d4 to ff86f95 Compare September 28, 2020 23:41
Adds an off-by-default `std` feature which activates the `std` features
of all of the other dependencies.

This is a somewhat lousy way of doing this, and I wish that there were a
better approach, so much that I posted a Pre-Pre-RFC about it:

https://internals.rust-lang.org/t/pre-pre-rfc-weak-cargo-feature-activation/13141

For now though, this is immediately useful to me.
@tarcieri tarcieri force-pushed the cryptography/std-feature branch from ff86f95 to d162162 Compare September 28, 2020 23:48
Copy link
Member

@newpavlov newpavlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, something like weak features would be really great to have.

@tarcieri tarcieri merged commit 25f2064 into master Sep 30, 2020
@tarcieri tarcieri deleted the cryptography/std-feature branch September 30, 2020 02:00
@tarcieri tarcieri mentioned this pull request Sep 30, 2020
dns2utf8 pushed a commit to dns2utf8/traits that referenced this pull request Jan 24, 2023
This version added a check that encoded Base64 is canonical, i.e. that
the decoded value will round-trip back to the encoded value.

Some of the test vectors did not have this property and have been fixed.
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.

2 participants