Skip to content

Depend on rust-crypto without default features, fixes #128 - #129

Open
middagj wants to merge 2 commits into
1Password:mainfrom
middagj:crypto-no-default-features
Open

middagj wants to merge 2 commits into
1Password:mainfrom
middagj:crypto-no-default-features

Conversation

@middagj

@middagj middagj commented Oct 26, 2026 •

Copy link
Copy Markdown

This resolves #128 by always adding rust-crypto without default features in other crates.

Another solution could be to not add rust-crypto to the default features and let the user explicitly chose. If the user chooses neither, it won't have an available backend.

@Progdrasil

Copy link
Copy Markdown
Collaborator

Thanks for the contribution, please include a CHANGELOG entry for this and it should be good to go

@middagj

middagj commented Oct 27, 2026

Copy link
Copy Markdown
Author

Thanks for the quick response. I added an entry in the CHANGELOG in 1b82fb9

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.

rust-crypto libraries are always added due to feature unification

2 participants