-
Notifications
You must be signed in to change notification settings - Fork 73
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
ERC4337 module feature: Extract execution to a separate function and require success in the internal call #95
ERC4337 module feature: Extract execution to a separate function and require success in the internal call #95
Conversation
6c3e5f5
to
f565930
Compare
4337/contracts/EIP4337Module.sol
Outdated
address entryPoint = _msgSender(); | ||
require(entryPoint == supportedEntryPoint, "Unsupported entry point"); | ||
|
||
(bool success, ) = msg.sender.call(executionData); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you think we'd benefit from an additional check for the execTransactionFromModule
signature?
f565930
to
aed12a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The approach should work and be gas efficient, would just add 2 checks
aed12a3
to
e381527
Compare
9cfd4dc
to
b40d9c6
Compare
b40d9c6
to
907e849
Compare
ad89aa2
to
02c1a14
Compare
02c1a14
to
ca698b0
Compare
This PR:
execTransactionFromModule
and had to be enabled as a module as wellsuccess
and revert on failing transactions to enable a proper gas estimationThis PR is a prototype only at the moment, and the goal is to seek the team's feedback on the approach.