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

Impl VerifyingKey::is_weak #277

Merged
merged 2 commits into from
Jan 31, 2023
Merged

Conversation

rozbb
Copy link
Contributor

@rozbb rozbb commented Jan 29, 2023

verify_strict does this check, but it might be useful to users, especailly those of verify_batch, to do this check before verification. The README in #275 refers users to this method.

Partially resolves #188

Also note that this solution might be overridden by a more principled approach to strict verification—one that follows the algorithms specified in this paper.

@rozbb rozbb requested a review from tarcieri January 29, 2023 01:37
@rozbb rozbb merged commit 5190ad6 into dalek-cryptography:release/2.0 Jan 31, 2023
rozbb added a commit to rozbb/ed25519-dalek that referenced this pull request Jan 31, 2023
rozbb added a commit that referenced this pull request Feb 2, 2023
* Added items to changelog for 2.0 release

* Removed unnecessary uses of std in doctests

* Gated `Context` behind `digest`

* Fixed noncompiling doctest when only `digest` is enabled

* README feature flag list mostly done

* Copied changelog to readme

* Redid the malleability section in README

* Added CONTRIBUTING.md

* Bumped version number to 2.0.0-pre.0; small changes to README

* Updated changelog for #277

* Added pem feature description

Co-authored-by: pinkforest(she/her) <[email protected]>
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