-
Notifications
You must be signed in to change notification settings - Fork 22
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
CHIP-0036: keccak256 CLVM operator #131
Conversation
This CHIP is now a |
Hi! Thank you for considering my request for the I wanted to add some context for the broader community. After this thread, the idea of an EIP-712-compatible Chia puzzle stuck with me. In short, a puzzle could likely be built in such a way that coins are controlled via EIP-712 signatures. EIP-712 is a widely-used signature standard on Ethereum that has significant wallet adoption - meaning that you could sign messages in virtually every Ethereum wallet. The exciting opportunity here is to sign messages with Ledger/Trezor devices or other hardware wallets - if it works, I think this is the best path forward for hardware wallet support for Chia. As mentioned in the CHIP description, the operator has been included in a previous (abandoned) CHIP. The code required to support it is, to my knowledge, still available in a non-merged branch. I've committed to fully testing this idea by building the puzzle mentioned above and releasing some tests for it as soon as the chialisp compiler/Rust simulator support |
Here is the CLVM implementation for this CHIP: |
Hey, all! I got my PoC working yesterday - the keccak operator can be used to enable hardware wallet support. The PoC repository is here, and a quick explanation can be found here. I still have to add soft fork support to the puzzle (the current one assumes keccak256 operator was hard-forked), which I'll probably do next week. |
This CHIP is now in Please review the CHIP at your convenience by leaving a comment in this PR. |
The block height when this CHIP's soft fork will activate is |
This CHIP is now in |
This CHIP is now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
No description provided.