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

Support for customising EIP 712 signature request #77

Closed
bhargav55 opened this issue Apr 6, 2021 · 2 comments
Closed

Support for customising EIP 712 signature request #77

bhargav55 opened this issue Apr 6, 2021 · 2 comments

Comments

@bhargav55
Copy link

Lot of parameters in EIP 712 signature are not required and need to add missing parameters to the signature request, can we have the option to customise the signature request.

@tomarsachin2271
Copy link
Collaborator

Support for custom TrustedForwarder is in the roadmap where developers will be able to deploy and use their own TrustedForwarder hence their own signature request. Can't comment on the timeline right now but it might take couple of months for this customisation feature.

@livingrock7
Copy link
Contributor

Yes, we will be adding customisation option for signature request and use minimal forwarder of your choice/provided by us.

EIP2771 only has two unnecessary paramters in some scenarios, which are token address and tokenGasPrice. Rest are definitely required. More on this...

Biconomy Forwarder uses ERC20ForwardRequest typehash. so we have generic forwarder eip2771 and forward (pay in erc20 tokens) For forward call is routed to ERC20Forwarder which in turn uses BiconomyForwarder to forward to recipient so thats the flow currently.

But for just using EIP 2771 we can deploy minimal generic forwarder. This would just remove need of passing token address and tokenGasPrice for forwarded transactions that don't need it.
Check this for example : https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/metatx/MinimalForwarder.sol

Closing this issue. We will update on the timeline for customisation feature. Feel free to open a PR on https://github.com/bcnmy/mexa for adding any contracts that enable customised typehash for signatures :)

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

No branches or pull requests

3 participants