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

Implementation of SimplPedPoP #106

Closed
wants to merge 28 commits into from
Closed

Conversation

Fiono11
Copy link

@Fiono11 Fiono11 commented May 6, 2024

This PR is an improvement of #102 based on feedback from #105.

Improvements

Add tests

Fix tests

Return keys from simplpedpop_recipient_all instead of points and scalars

Replace pub(crate) with pub(super)

Fix in ciphertexts loop

Add wrapper types

Implement SecretShare and EncryptedSecretShare types

Implement SecretPolynomial and PolynomialCommitment types

Restructuring of files

Remove derive_key_from_scalar

Remove ephemeral key

Add Identifier type

Add identifiers to dkg_output

Remove proof of possession signature

Improvements

Fix deserialization of dkg output

Remove unwrap

Improve errors

Add polynomial tests

Reimplement ephemeral key and proof of possession

Merge changes from final_frost branch to simplpedpop

Undo formatting
@burdges
Copy link
Collaborator

burdges commented Jul 30, 2024

This'll be merged under #110

Thank you! Sorry for the crazy delays on my part. :(

@burdges burdges closed this Jul 30, 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