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

no_std compatibility #3512

Closed
1 of 3 tasks
redshiftzero opened this issue Dec 12, 2023 · 5 comments
Closed
1 of 3 tasks

no_std compatibility #3512

redshiftzero opened this issue Dec 12, 2023 · 5 comments

Comments

@redshiftzero
Copy link
Member

redshiftzero commented Dec 12, 2023

For embedded devices such as hardware wallets, we'll need no_std compatibility for at least:

@cronokirby
Copy link
Contributor

One thing to note is that we don't need to full functionality suite for penumbra-keys. e.g. we don't need to construct transaction with key agreement, we just need to sign, export a full viewing key, and derive addresses. Which, admittedly, is 90% of the code.

@hdevalence
Copy link
Member

It's maybe worse than that, no? The custodian has to process the TransactionPlan into an EffectHash.

@cronokirby
Copy link
Contributor

I had a chat with @redshiftzero earlier today and we sort of realized that the logic for doing this would need to be extracted out most likely and rewritten in an embedded context, leaning on the fact that effect hashes are calculated from the proto encoding. We should probably write up the thinking here somewhere.

@hdevalence
Copy link
Member

Originally we thought we'd need to do something like that to have no_std support for WASM to run code in the browser. Then it turned out we didn't. So while it might be the case, we should probably not rewrite large parts of our crates without actually gathering requirements first. I don't think that's happened yet. It would be helpful to hear from the Zondax team about how things have gone so far.

@redshiftzero
Copy link
Member Author

Closing this in favor of the plan in #3526

@github-project-automation github-project-automation bot moved this to Testnet 63: Rhea (Web Wallet) in Testnets Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Testnet 63: Rhea (Web Wallet)
Development

No branches or pull requests

3 participants