Add ERC-7677 Paymaster service for gasless transactions #2815
Conversation
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
Migrate EIP-7702 from manual signing (broken due to eth_sign disabled in MetaMask) to wallet_sendCalls with Paymaster sponsorship. Changes: - Add PaymasterModule with ERC-7677 JSON-RPC endpoints (pm_getPaymasterStubData, pm_getPaymasterData) - Add deposit address validation to only sponsor transfers to DFX deposit addresses - Update sell.service.ts and swap.service.ts to set usePaymaster and paymasterUrl fields - Update UnsignedTxDto with new Paymaster fields - Add isValidDepositAddress method to DepositService The Paymaster validates that transactions are only to valid DFX deposit addresses, preventing abuse of gas sponsorship.
5 tasks
- PaymasterService tests (20 tests): - pm_getPaymasterStubData and pm_getPaymasterData handling - Chain validation (Ethereum, Arbitrum, Base, Sepolia, etc.) - Transfer recipient extraction (ERC20, ERC-7579) - Deposit address validation - Error handling - PaymasterController tests (8 tests): - Request/response passthrough - Chain ID parameter handling - Error propagation
Collaborator
Author
|
Closing: Implementation requires Pimlico integration (or similar ERC-4337 paymaster provider) to function. Current backend is a stub that won't work with MetaMask's wallet_sendCalls. See analysis: services/docs/PIMLICO-PAYMASTER-ANALYSIS.md |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
PaymasterModulewith ERC-7677 JSON-RPC endpoints (pm_getPaymasterStubData,pm_getPaymasterData)sell.service.tsandswap.service.tsto setusePaymasterandpaymasterUrlfields when user has 0 gasUnsignedTxDtowith new Paymaster fieldsisValidDepositAddressmethod toDepositServiceContext
The original EIP-7702 implementation using manual
eth_signis broken because MetaMask disableseth_signby default. This PR migrates towallet_sendCalls(ERC-5792) with Paymaster sponsorship.New Flow
MetaMask handles EIP-7702 internally, and the Paymaster sponsors gas for transactions to valid DFX deposit addresses.
Security
toaddress is a registered DFX deposit addressDependencies
Test plan