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

Structure message signing EIP-712 #831

Closed
5 tasks done
gfukushima opened this issue Jul 3, 2023 · 1 comment · Fixed by #893
Closed
5 tasks done

Structure message signing EIP-712 #831

gfukushima opened this issue Jul 3, 2023 · 1 comment · Fixed by #893
Assignees
Labels
TeamCerberus Under active development by TeamCerberus @Consensys

Comments

@gfukushima
Copy link
Contributor

gfukushima commented Jul 3, 2023

Implement structured message signing RPC eth_signTypedData

"Aim of EIP-712 is to simplify off-chain message signing for efficient on-chain verification"

The method eth_signTypedData is added to the Ethereum JSON-RPC. The method parallels eth_sign.

(NB: it's not actually in the spec https://github.com/ethereum/execution-apis/blob/main/src/eth/sign.yaml)

Tasks

Resources

@gfukushima gfukushima added the TeamCerberus Under active development by TeamCerberus @Consensys label Jul 3, 2023
@siladu siladu self-assigned this Jul 13, 2023
@gfukushima gfukushima self-assigned this Aug 27, 2023
@jframe jframe added the doc-change-required Indicates an issue or PR that requires doc to be updated label Aug 29, 2023
@gfukushima gfukushima mentioned this issue Aug 30, 2023
3 tasks
@siladu
Copy link
Contributor

siladu commented Aug 30, 2023

Check that web3j implementation is up to date (uses eth_signTypedData_v4?)

@gfukushima what was the outcome of this? Which version is web3j using?

@alexandratran alexandratran removed the doc-change-required Indicates an issue or PR that requires doc to be updated label Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TeamCerberus Under active development by TeamCerberus @Consensys
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants