Skip to content

feat: CRP-2797 Add functions to access the production master key#171

Merged
randombit merged 9 commits intomainfrom
jack/crp-2797
Jun 30, 2025
Merged

feat: CRP-2797 Add functions to access the production master key#171
randombit merged 9 commits intomainfrom
jack/crp-2797

Conversation

@randombit
Copy link
Copy Markdown
Contributor

This allows performing key derivation completely offline, without having to interact with the IC.

This allows performing key derivation completely offline, without
having to interact with the IC.
@randombit randombit requested a review from a team June 27, 2025 17:33
Comment thread frontend/ic_vetkeys/src/utils/utils.test.ts Outdated
Copy link
Copy Markdown
Contributor

@altkdf altkdf left a comment

Choose a reason for hiding this comment

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

LGTM @randombit ! Just one question. If we decide to have the same API in TS as in Rust, we could also do that in another PR. Maybe we should add test_key_1 in this PR as well though?

@randombit
Copy link
Copy Markdown
Contributor Author

Maybe we should add test_key_1 in this PR as well though?

Done. I also added a test for test_key_1 derivation that was generated using a canister on mainnet.

@randombit
Copy link
Copy Markdown
Contributor Author

Ah, nice. I realized key_1 can already be used for deriving public keys, so I updated the tests for that to also use live data.

Copy link
Copy Markdown
Contributor

@altkdf altkdf left a comment

Choose a reason for hiding this comment

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

Cool, thanks @randombit ! Please alsoo add to the changelog.

@randombit randombit merged commit dc40657 into main Jun 30, 2025
34 checks passed
@randombit randombit deleted the jack/crp-2797 branch June 30, 2025 13:15
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