Skip to content

Specify precisely the data to be signed#23

Merged
daira merged 2 commits into
masterfrom
529.change-signature-design.0
Mar 28, 2016
Merged

Specify precisely the data to be signed#23
daira merged 2 commits into
masterfrom
529.change-signature-design.0

Conversation

@daira
Copy link
Copy Markdown
Collaborator

@daira daira commented Mar 16, 2016

This includes a design change to use a single signature covering all of the transaction except pourPubKey and pubSig.

Signed-off-by: Daira Hopwood daira@jacaranda.org

daira added 2 commits March 16, 2016 01:36
to use a single signature covering all of the transaction except
pourPubKey and pubSig.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>
Signed-off-by: Daira Hopwood <daira@jacaranda.org>
@ebfull
Copy link
Copy Markdown
Collaborator

ebfull commented Mar 18, 2016

ephemeralKey in your table describing the contents of a PourTx needs to be magenta.

Also, pourSig/pourPubKey should be specific to the pour because the pours may be created by different people.

@ebfull
Copy link
Copy Markdown
Collaborator

ebfull commented Mar 20, 2016

Besides flexibility, it's also nice that CPourTx doesn't need any outside context to verify the zkSNARK. If we had shared, common fields in lower levels of the transaction it complicates the story.

@daira daira merged commit 0e6953a into master Mar 28, 2016
@daira
Copy link
Copy Markdown
Collaborator Author

daira commented Mar 29, 2016

We decided to use the single-signature approach that is in the current spec.

@daira daira deleted the 529.change-signature-design.0 branch April 21, 2018 13:14
daira pushed a commit to daira/zips that referenced this pull request Nov 8, 2023
As per the title, this is a tiny fix to adjust the boundary of the
math environment which wasn't properly closed and thus rendered poorly
on the html.
daira pushed a commit to daira/zips that referenced this pull request Feb 7, 2024
As per the title, this is a tiny fix to adjust the boundary of the
math environment which wasn't properly closed and thus rendered poorly
on the html.
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