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

Validate JoinSplit proofs #322

Closed
dconnolly opened this issue Mar 25, 2020 · 1 comment · Fixed by #3128
Closed

Validate JoinSplit proofs #322

dconnolly opened this issue Mar 25, 2020 · 1 comment · Fixed by #3128
Assignees
@conradoplg
Labels
NU Sprout Network Upgrade: Sprout specific tasks (before Overwinter)
Milestone
2021 Sprint 24

Comments

@dconnolly
Copy link
Contributor

dconnolly commented Mar 25, 2020 •
edited
Loading

image

https://zips.z.cash/protocol/protocol.pdf#joinsplitstatement

Luckily we can use the Sprout JoinSplits in old transaction/block test vectors to exercise this.
@dconnolly dconnolly added this to the Validate transactions. milestone Mar 25, 2020
@hdevalence hdevalence mentioned this issue Mar 25, 2020
Closed
@dconnolly dconnolly changed the title Validate JoinSplit proof Validate JoinSplit proofs Mar 25, 2020
@hdevalence
Copy link
Contributor

xref: #305 (comment)

@dconnolly dconnolly self-assigned this Jun 4, 2020
@mpguerra mpguerra removed this from the Transaction Validation milestone Jan 5, 2021
@mpguerra mpguerra added the NU Sprout Network Upgrade: Sprout specific tasks (before Overwinter) label Jan 15, 2021
@dconnolly dconnolly mentioned this issue Mar 5, 2021
Closed
53 tasks
@mpguerra mpguerra mentioned this issue Mar 10, 2021
Closed
13 tasks
@mpguerra mpguerra removed the E-med label Mar 23, 2021
@mpguerra mpguerra added this to the 2021 Sprint 22 milestone Oct 1, 2021
@mpguerra mpguerra assigned dconnolly and teor2345 and unassigned dconnolly Nov 8, 2021
This was referenced Nov 15, 2021
Merged
Closed
Merged
Merged
@mpguerra mpguerra assigned conradoplg and unassigned teor2345 Nov 22, 2021
This was referenced Dec 1, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@conradoplg conradoplg

Labels
NU Sprout Network Upgrade: Sprout specific tasks (before Overwinter)
Projects
None yet
Milestone
2021 Sprint 24
Development

Successfully merging a pull request may close this issue.

Validate JoinSplit proofs ZcashFoundation/zebra
5 participants
@conradoplg @hdevalence @dconnolly @mpguerra @teor2345