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

Update eth_signTypedData_v4 example #274

Merged
merged 2 commits into from
Nov 22, 2023
Merged

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Nov 21, 2023

The example eth_signTypedData_v4 message has been updated to include a field that can reproduce this bug: MetaMask/metamask-mobile#7792 That signing bug results in invalid signatures for any messages that include a bytes field with the value 0x.

The example `eth_signTypedData_v4` message has been updated to include
a field that can reproduce this bug: MetaMask/metamask-mobile#7792
That signing bug results in invalid signatures for any messages that
include a `bytes` field with the value `0x`.
Copy link
Collaborator

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Looks good.

Copy link
Contributor

@legobeat legobeat left a comment

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt Gudahtt merged commit e4778fc into main Nov 22, 2023
5 checks passed
@Gudahtt Gudahtt deleted the add-0x-to-sign-typed-data-examples branch November 22, 2023 00:26
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.

3 participants