fix(eip7702): validate factory field with eip7702 requirements#712
Merged
mouseless0x merged 9 commits intomainfrom Apr 7, 2026
Merged
fix(eip7702): validate factory field with eip7702 requirements#712mouseless0x merged 9 commits intomainfrom
mouseless0x merged 9 commits intomainfrom
Conversation
- Add validation for 0x7702 factory address in both short and padded forms - Require eip7702Auth when factory is set to 0x7702 - Add comprehensive test coverage for both factory address formats
plusminushalf
approved these changes
Mar 31, 2026
- Add early validation check for EIP-7702 userOps with 0x7702 factory - Ensure eip7702Auth is provided when factory is 0x7702 - Update error message for consistency
- Removed validation check that incorrectly rejected userOps with factory 0x7702 but no eip7702Auth - This check was overly restrictive and not aligned with EIP-7702 specification
- Add early userOp field validation in eth_estimateUserOperationGas - Add isEstimation flag to skip gas checks during estimation - Consolidate EIP-7702 factory validation and remove duplication
- Add test cases for rejecting estimation with 0x7702 factory without eip7702Auth - Test both short form (0x7702) and zero-padded form validation - Skip test for v0.6 which doesn't support factory field
…tion - Separate bundler client creation for gas estimation calls - Rename smartAccountClient for clarity - Pass entryPointAddress explicitly to estimateUserOperationGas
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add validation for 0x7702 factory address in both short and padded
forms
Require eip7702Auth when factory is set to 0x7702
Add comprehensive test coverage for both factory address formats