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

Proposal to include EIP-3074 in London #260

Closed
SamWilsn opened this issue Mar 2, 2021 · 12 comments
Closed

Proposal to include EIP-3074 in London #260

SamWilsn opened this issue Mar 2, 2021 · 12 comments

Comments

@SamWilsn
Copy link

SamWilsn commented Mar 2, 2021

EIP here: Sponsored Transaction Precompile

Simple Summary

Creates two new EVM instructions that authorize (via an ECDSA signature) a contract to act on behalf of an externally owned account.

@timbeiko
Copy link
Collaborator

timbeiko commented Mar 2, 2021

Thanks for proposing @SamWilsn! I've added it to the agenda for this Friday's call, but it's already quite packed, so there is a risk we don't have time to get to it. If you'd rather not attend the entire call with the risk of your EIP not being discussed, let me know. I can add it to the call after and make sure we prioritize it then.

@SamWilsn
Copy link
Author

SamWilsn commented Mar 2, 2021

I don't mind attending this week regardless. The calls are always a mix of entertaining and enlightening!

@timbeiko
Copy link
Collaborator

timbeiko commented Mar 2, 2021

It remains "optimistically scheduled", then :-)

@adietrichs
Copy link
Member

I created a (relatively) simple example for what an invoker contract for EIP-3074 could look like:
https://gist.github.com/adietrichs/ab69fa2e505341e3744114eda98a05ab

@timbeiko
Copy link
Collaborator

timbeiko commented Mar 5, 2021

@SamWilsn added to next call's agenda given we couldn't cover it today https://github.com/ethereum/pm/blob/master/README.md#acd-108-march-19-2021

@timbeiko
Copy link
Collaborator

From the ETH R&D discord:

Alexey, TurboGeth:

Sponsored Transaction Precompile - not to include to London - is there a way to do it without precompile (these are ugly things and should not be regarded as normal artefacts of EVM development), also consider who exactly will need it and what are the alternatives.

Martin, Geth:

3074 Should not be included. It's still iterated on, and the semantics are very hard to overview. The possible quirks with new call types are many both on infra layer and app layer, and I think it needs more time to be fully understood.

@shemnon
Copy link
Contributor

shemnon commented Mar 15, 2021

I would want to see how this interacts with account abstraction before I would be comfortable moving forward with it. I concur with alexey and martin in that it is too early in the design to freeze it for implementation, which moves it out of London.

@chfast
Copy link
Member

chfast commented Mar 15, 2021

FYI, we have an issue with terms in this EIP. It does not propose any new procompile. It adds a new call instruction + one more new instruction related to this new call kind.

@SamWilsn
Copy link
Author

I would want to see how this interacts with account abstraction before I would be comfortable moving forward with it. I concur with alexey and martin in that it is too early in the design to freeze it for implementation, which moves it out of London.

In the interest in centralizing discussion, I've replied on the ethereum-magicians thread.

@SamWilsn
Copy link
Author

FYI, we have an issue with terms in this EIP. It does not propose any new procompile. It adds a new call instruction + one more new instruction related to this new call kind.

I updated the description of this issue to match the latest EIP. I hope that's what you're referring to?

@timbeiko
Copy link
Collaborator

timbeiko commented Apr 2, 2021

The threat analysis document that was mentioned on today's AllCoreDevs: https://hackmd.io/@matt/BknnAnyNu

@timbeiko
Copy link
Collaborator

We agreed on ACD111 to not include this in London, so closing this.

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

5 participants