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

eth_signTypedData incompatible with ethers _signTypedData #1199

Closed
thegostep opened this issue Jan 24, 2021 · 5 comments · Fixed by #1189
Closed

eth_signTypedData incompatible with ethers _signTypedData #1199

thegostep opened this issue Jan 24, 2021 · 5 comments · Fixed by #1189
Assignees
Labels
type:bug Something isn't working

Comments

@thegostep
Copy link

const permission = await ethers.provider
    .getSigner(ownerAddress)
    ._signTypedData(domain, types, value)

image

@ricmoo
Copy link

ricmoo commented Jan 24, 2021

Does hard hat implement the eth_signTypedData_v4 JSON-RPC method?

@thegostep
Copy link
Author

Their docs show they support eth_signTypedData

https://hardhat.org/hardhat-network/#supported-methods

@thegostep
Copy link
Author

@ricmoo
Copy link

ricmoo commented Jan 24, 2021

I don’t think that call is final yet, so they may need to at the very least make an alias to the _v4 variant that other clients use. Otherwise it is too hard to tell what parameters to pass to the function. Ethers only supports the v4 parameters description and has no intention of supporting legacy versions of EIP-712...

Do you know what parameter description they are using for that call? Is it v4 compatible?

@ricmoo
Copy link

ricmoo commented Jan 24, 2021

Their string matching against the method name is wrong though, as it is checking for eth_signTypedData. So that if condition is skipped.

thegostep added a commit to thegostep/buidler that referenced this issue Jan 24, 2021
@scnale scnale linked a pull request Jan 25, 2021 that will close this issue
@scnale scnale added the type:bug Something isn't working label Jan 25, 2021
@fvictorio fvictorio self-assigned this Nov 27, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type:bug Something isn't working
Projects
None yet
4 participants