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

Support tagged ZIP 32 child derivation for registered application protocols #24

Merged
merged 8 commits into from
Feb 20, 2025

Conversation

daira
Copy link
Contributor

@daira daira commented Feb 12, 2025

This implements the specification in zcash/zips#978 .

@daira daira requested review from str4d and nuttycom February 12, 2025 20:10
@daira daira force-pushed the support-tagged-child-derivation branch from cdd24c0 to 402a3e4 Compare February 13, 2025 03:14
@daira daira changed the title Support tagged ZIP 32 child derivation in zip32 Support tagged ZIP 32 child derivation for registered application protocols Feb 13, 2025
@daira daira force-pushed the support-tagged-child-derivation branch from 402a3e4 to 3999336 Compare February 13, 2025 03:18
@daira daira force-pushed the support-tagged-child-derivation branch 3 times, most recently from c788e52 to 7b12eaf Compare February 15, 2025 02:24
…ified

in a ZIP.

Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
@daira daira force-pushed the support-tagged-child-derivation branch from 7b12eaf to c7442d7 Compare February 17, 2025 21:22
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
@daira daira force-pushed the support-tagged-child-derivation branch from 04b84b3 to e5b4ea2 Compare February 18, 2025 00:25
daira and others added 4 commits February 18, 2025 00:39
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
Copy link
Contributor Author

@daira daira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK on @str4d's commits.

@daira daira force-pushed the support-tagged-child-derivation branch 5 times, most recently from 21d9e79 to 3003f3f Compare February 20, 2025 02:43
@daira daira force-pushed the support-tagged-child-derivation branch from 011bf64 to 8ba2e44 Compare February 20, 2025 03:14
str4d
str4d previously requested changes Feb 20, 2025
Copy link
Collaborator

@str4d str4d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 8ba2e44, except that I have not checked the test vectors are correctly copied from the test vectors repo.

@daira daira force-pushed the support-tagged-child-derivation branch from 8ba2e44 to 6bc0ab6 Compare February 20, 2025 05:56
@daira daira requested a review from str4d February 20, 2025 06:03
@daira daira dismissed str4d’s stale review February 20, 2025 06:04

requested changes addressed

@daira daira force-pushed the support-tagged-child-derivation branch from 6bc0ab6 to 23e0fb3 Compare February 20, 2025 06:05
@daira
Copy link
Contributor Author

daira commented Feb 20, 2025

Ready for re-review.

@daira daira force-pushed the support-tagged-child-derivation branch from 23e0fb3 to 354834f Compare February 20, 2025 06:12
Copy link
Collaborator

@str4d str4d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 354834f, modulo I still have not checked the test vectors match yet.

Copy link
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK fb73395

daira and others added 2 commits February 20, 2025 19:37
`registered::SecretKey::{derive_child, derive_child_cryptovalue}`.

Co-authored-by: Jack Grigg <jack@electriccoin.co>
Co-authored-by: Kris Nuttycombe <kris@nutty.land>
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
Co-authored-by: Kris Nuttycombe <kris@nutty.land>
Signed-off-by: Daira-Emma Hopwood <daira@jacaranda.org>
@daira daira force-pushed the support-tagged-child-derivation branch from fb73395 to 2c177cb Compare February 20, 2025 19:41
Copy link
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-utACK 2c177cb

@nuttycom
Copy link
Contributor

I have verified that the test vectors were correctly imported from zcash/zcash-test-vectors

@nuttycom nuttycom merged commit 6632f2c into zcash:master Feb 20, 2025
10 checks passed
@daira daira deleted the support-tagged-child-derivation branch February 20, 2025 20:52
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.

3 participants