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.

Sign up for GitHub

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

Jump to bottom

Implement EIP-5267 Retrieval of EIP-712 domain #3968

Closed
frangio opened this issue Jan 18, 2023 · 0 comments · Fixed by #3969
Closed

Implement EIP-5267 Retrieval of EIP-712 domain #3968

frangio opened this issue Jan 18, 2023 · 0 comments · Fixed by #3969
Milestone
4.9

Comments

@frangio
Copy link
Contributor

frangio commented Jan 18, 2023

EIP-5267 is in Last Call and should be Final by the time we release 4.9.

I think it should be added by default in EIP712, it doesn't add runtime cost and my opinion is that it should always be used as it makes EIP-712 signatures more usable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.9
Development

Successfully merging a pull request may close this issue.

Include EIP-5267 discovery in EIP-712 Amxx/openzeppelin-contracts
1 participant
@frangio