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

Add Ristretto255 support #106

Merged
merged 3 commits into from
May 2, 2022
Merged

Add Ristretto255 support #106

merged 3 commits into from
May 2, 2022

Conversation

Traderjoe95
Copy link
Contributor

libsodium is supporting the Ristretto255 prime order group. This PR implements support in LazySodium.

@gurpreet-
Copy link
Contributor

Libsodium has Ristretto support now? 🙂

Reviewing now.

@gurpreet- gurpreet- self-requested a review July 3, 2021 10:37
Copy link
Contributor

@gurpreet- gurpreet- left a comment

Choose a reason for hiding this comment

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

Thank you for your hard work in creating this perfect PR!

I have some comments. Let's work on them together

@Traderjoe95
Copy link
Contributor Author

Hey, sorry for the late reply, I've been having quite some busy days lately. I will address your suggested changes this week as soon as I have time 😄

@michielgkalkman
Copy link

michielgkalkman commented Mar 17, 2022

Hi, any chance this will be merged and deployed any time soon?

@anhepol
Copy link

anhepol commented Mar 31, 2022

As @michielgkalkman says. @Traderjoe95 It would be great if this pull request could be completed. :)

@Traderjoe95
Copy link
Contributor Author

Hey all, sorry for the massive delay. I've been completely swamped at work lately, so I couldn't really continue working on this. But I guess I will be able to finish all those changes today. Let's see.

@Traderjoe95
Copy link
Contributor Author

@gurpreet- I addressed all your comments, I hope we are clear to merge now. 😃

Cheers!

@gurpreet- gurpreet- merged commit 92a0b54 into terl:master May 2, 2022
@gurpreet-
Copy link
Contributor

Looks great! Merged!

@michielgkalkman
Copy link

Thanks!!

@Traderjoe95 Traderjoe95 deleted the ristretto255 branch June 14, 2022 10:05
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.

4 participants