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

Import Poseidon hash into Zebra #2064

Closed
dconnolly opened this issue Apr 24, 2021 · 3 comments
Closed

Import Poseidon hash into Zebra #2064

dconnolly opened this issue Apr 24, 2021 · 3 comments
Labels
A-cryptography Area: Cryptography related A-rust Area: Updates to Rust code NU-5 Network Upgrade: NU5 specific tasks

Comments

@dconnolly
Copy link
Contributor

dconnolly commented Apr 24, 2021

As defined in NU5 spec:
https://zips.z.cash/protocol/nu5.pdf#poseidonhash

For reference, one implementation is over in the orchard crate: https://github.com/zcash/orchard/blob/main/src/primitives/poseidon.rs

Scheduling

This is only required for the wallet work, because it's used to create nullifiers.

@dconnolly dconnolly added A-rust Area: Updates to Rust code S-needs-triage Status: A bug report needs triage NU-5 Network Upgrade: NU5 specific tasks P-Medium labels Apr 24, 2021
@dconnolly dconnolly self-assigned this Apr 24, 2021
@mpguerra mpguerra mentioned this issue Apr 26, 2021
53 tasks
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Apr 27, 2021
@mpguerra mpguerra added this to the 2021 Sprint 12 milestone May 10, 2021
@teor2345
Copy link
Contributor

I moved this task to sprint 13 because the anchors are a higher priority - they are blocking the history chain contextual validation and testing.

@teor2345 teor2345 changed the title Implement or pull in Poseidon hash Import Poseidon hash into Zebra Jun 23, 2021
@teor2345
Copy link
Contributor

We decided to favour importing cryptography over re-implementing it, at least until after NU5 activation.

@mpguerra mpguerra removed this from the 2021 Sprint 15 milestone Jul 26, 2021
@mpguerra mpguerra added this to the 2021 Sprint 22 milestone Oct 1, 2021
@mpguerra mpguerra removed this from the 2021 Sprint 22 milestone Oct 13, 2021
@conradoplg conradoplg added the A-cryptography Area: Cryptography related label Jun 1, 2022
@mpguerra mpguerra moved this to 🆕 New in Zebra Sep 22, 2022
@mpguerra mpguerra added this to Zebra Sep 22, 2022
@teor2345
Copy link
Contributor

We're not planning to do full-node wallet work any time soon.

If we do, we'll need to decide if this ticket is in scope, and update its content, because the spec has changed since it was created.

@teor2345 teor2345 closed this as not planned Won't fix, can't repro, duplicate, stale Sep 28, 2022
Repository owner moved this from 🆕 New to ✅ Done in Zebra Sep 28, 2022
@mpguerra mpguerra moved this from ✅ Done to 🛑 Won't Fix in Zebra Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cryptography Area: Cryptography related A-rust Area: Updates to Rust code NU-5 Network Upgrade: NU5 specific tasks
Projects
Archived in project
Development

No branches or pull requests

4 participants