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.

Sign up for GitHub

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

Jump to bottom

Change JoinSplit signatures to use Ed25519 #51

Closed
daira opened this issue May 30, 2016 · 6 comments
Closed

Change JoinSplit signatures to use Ed25519 #51

daira opened this issue May 30, 2016 · 6 comments
Assignees
@daira
Labels
protocol spec
Milestone
Spec ready for audit

Comments

@daira
Copy link
Collaborator

daira commented May 30, 2016

See zcash/zcash#977
@nathan-at-least nathan-at-least mentioned this issue May 30, 2016
Closed
@ebfull
Copy link
Contributor

ebfull commented May 30, 2016

Also, keep in mind because the ed25519 public keys are 256 bits we do not need to hash them before we use them in the h_sig calculation.

@daira
Copy link
Collaborator Author

daira commented May 30, 2016

I'm in the middle of a big refactoring of the spec to address #27, which involves a lot of movement of LaTeX code between sections, so I'll finish that first to avoid a complicated merge/rebase.

@daira daira self-assigned this May 30, 2016
@daira
Copy link
Collaborator Author

daira commented May 31, 2016

Reminder to self: bump the version to 2016-0.beta after making this change.

@zmanian
Copy link

zmanian commented Jun 27, 2016

I haven't been able to figure out from specification how the Join Split key is used in the Pour verification. I might be missing it or it might need to improved in the documention.

@daira daira added this to the Spec ready for audit milestone Aug 7, 2016
@daira
Copy link
Collaborator Author

daira commented Sep 4, 2016

Fixed in 2016.0-beta-1.

@daira daira closed this as completed Sep 4, 2016
@daira
Copy link
Collaborator Author

daira commented Sep 4, 2016

@zmanian: feel free to open a new issue if that's still unclear.

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

@daira daira

Labels
protocol spec
Projects
None yet
Milestone
Spec ready for audit
Development

No branches or pull requests
3 participants
@zmanian @daira @ebfull