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

ECDSA recover #4

Merged
merged 9 commits into from Aug 4, 2017

Conversation

Projects
None yet
2 participants
@pirapira
Contributor

pirapira commented Aug 4, 2017

This PR adds a module RecoverableSign that provides the functionalities declared in secp256k1_recovery.h.

pirapira added some commits Aug 3, 2017

@dakk dakk merged commit c8619e6 into dakk:master Aug 4, 2017

@pirapira pirapira deleted the pirapira:ecdsa_recover branch Aug 4, 2017

@pirapira

This comment has been minimized.

Show comment
Hide comment
@pirapira

pirapira Aug 4, 2017

Contributor

@dakk will you push a new version to the opam repository? I need it for testing https://github.com/pirapira/eth-isabelle project because the sender of an Ethereum transaction can only be found using ECDSA recover.

Contributor

pirapira commented Aug 4, 2017

@dakk will you push a new version to the opam repository? I need it for testing https://github.com/pirapira/eth-isabelle project because the sender of an Ethereum transaction can only be found using ECDSA recover.

@pirapira

This comment has been minimized.

Show comment
Hide comment
@pirapira

pirapira Aug 4, 2017

Contributor

@dakk now I see you've already started. Thanks!

Contributor

pirapira commented Aug 4, 2017

@dakk now I see you've already started. Thanks!

@dakk

This comment has been minimized.

Show comment
Hide comment
@dakk

dakk Aug 4, 2017

Owner

@pirapira Yep, just submitted! Nice work

(I put your name / email in the opam author list and in Contributions section in the README.md)

Owner

dakk commented Aug 4, 2017

@pirapira Yep, just submitted! Nice work

(I put your name / email in the opam author list and in Contributions section in the README.md)

@pirapira

This comment has been minimized.

Show comment
Hide comment
@pirapira

pirapira Aug 4, 2017

Contributor

Thank you!

Contributor

pirapira commented Aug 4, 2017

Thank you!

@dakk

This comment has been minimized.

Show comment
Hide comment
@dakk
Owner

dakk commented Aug 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment