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

[bign256] Digital signature #935

Merged
merged 43 commits into from
Dec 14, 2023
Merged

[bign256] Digital signature #935

merged 43 commits into from
Dec 14, 2023

Conversation

makavity
Copy link
Contributor

@makavity makavity commented Oct 3, 2023

  • Implement tests
  • Implement signing
  • Implement verifying

Need a review, thanks :)

makavity and others added 30 commits March 22, 2023 10:07
Co-authored-by: Tony Arcieri <[email protected]>
Co-authored-by: Tony Arcieri <[email protected]>
Co-authored-by: Tony Arcieri <[email protected]>
Co-authored-by: Tony Arcieri <[email protected]>
Co-authored-by: Tony Arcieri <[email protected]>
Copy link
Member

@tarcieri tarcieri left a comment

Choose a reason for hiding this comment

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

FYI, I know little of this algorithm and can't really speak to the correctness of the implementation, but this seems reasonable enough

@tarcieri tarcieri merged commit b21dd4f into RustCrypto:master Dec 14, 2023
122 checks passed
@tarcieri tarcieri mentioned this pull request Jan 5, 2024
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