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

Allow 7702 addresses to recieve funds #308

Closed
lightclient opened this issue Oct 29, 2024 · 2 comments
Closed

Allow 7702 addresses to recieve funds #308

lightclient opened this issue Oct 29, 2024 · 2 comments

Comments

@lightclient
Copy link

Currently there is a ban on contract accounts receiving faucet funds based on this heuristic:

public checkIsContract(addr: string): Promise<boolean> {
return this.web3.eth.getCode(addr).then((res) => res && !!res.match(/^0x[0-9a-f]{2,}$/));
}

I think it should be updated to allow contracts with the 0xef0100 prefix.

@pk910
Copy link
Owner

pk910 commented Nov 3, 2024

Heya @lightclient,

I don't get why I need to whitelist the delegation prefix here?
As far as I understood 7702, this special code is only used while executing the transaction that holds the authorization and is cleared afterwards?
So the account should not have any code attached after execution completed, right?

If that's right, no account can ever have that code prefix for static calls (as it's not within a transaction execution context)..

@lightclient
Copy link
Author

No the code is set permanently!

@pk910 pk910 closed this as completed in dbe2f71 Nov 5, 2024
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

2 participants