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

Implement ZIP-216: Require Canonical Point Encodings #1798

Closed
dconnolly opened this issue Feb 22, 2021 · 1 comment · Fixed by #2351
Closed

Implement ZIP-216: Require Canonical Point Encodings #1798

dconnolly opened this issue Feb 22, 2021 · 1 comment · Fixed by #2351
Assignees
Labels
A-consensus Area: Consensus rule updates A-dependencies Area: Dependency file updates A-rust Area: Updates to Rust code NU-5 Network Upgrade: NU5 specific tasks

Comments

@dconnolly
Copy link
Contributor

dconnolly commented Feb 22, 2021

zcash/zips#400
https://zips.z.cash/zip-0216
zkcrypto/jubjub#43

Blocked on the jubjub crate implementing this, or we implement it ourselves.

@dconnolly dconnolly added NU-5 Network Upgrade: NU5 specific tasks A-consensus Area: Consensus rule updates A-dependencies Area: Dependency file updates A-rust Area: Updates to Rust code labels Feb 22, 2021
@dconnolly dconnolly mentioned this issue Mar 5, 2021
53 tasks
@mpguerra mpguerra added this to the 2021 Sprint 7 milestone Mar 26, 2021
@teor2345 teor2345 modified the milestones: 2021 Sprint 7, 2021 Sprint 9 Mar 31, 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.

@dconnolly should we re-estimate this task? I think it's only a dependency update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-consensus Area: Consensus rule updates A-dependencies Area: Dependency file updates A-rust Area: Updates to Rust code NU-5 Network Upgrade: NU5 specific tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants