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

EIP-2364: eth/64: forkid-extended protocol handshake #2365

Closed
karalabe opened this issue Nov 8, 2019 · 3 comments · Fixed by #5121
Closed

EIP-2364: eth/64: forkid-extended protocol handshake #2365

karalabe opened this issue Nov 8, 2019 · 3 comments · Fixed by #5121

Comments

@karalabe
Copy link
Member

karalabe commented Nov 8, 2019

This is a discussion thread for #2364


Abstract

The forkid (EIP-2124) was designed to permit two Ethereum nodes to quickly and cheaply decide if they are compatible or not, not only at a genesis/networking level, but also from the perspective of the currently passed network updates (i.e. forks).

EIP-2124 only defined how the forkid is calculated and validated, but due to the numerous possible applications, it did not itself spec out how the forkid should be exchanged between peers. This current EIP requests the inclusion of the forkid as a new field in the Ethereum wire protocol (eth) handshake (releasing a new version, eth/64).

By cross validating each others' forks already during the handshake, incompatible nodes can disconnect before expensive block exchanges and validations take place (PoW check, EVM execution, state reconstruction). This furher prevents peer slots from being taken up by nodes that are incompatible, but have not yet been detected as such.

@fjl
Copy link
Contributor

fjl commented Jan 7, 2020

Spec update: ethereum/devp2p#132

@lithp
Copy link

lithp commented Feb 26, 2020

I haven't been following along very closely, so I'm sorry that this is a dumb question!

But, I believe this spec is not only implemented but it's already deployed to the network. Why is this still in the Draft status? It ought to be Final, there's no changing it now!

I also have the same question for eth/65: #2465

@github-actions
Copy link

There has been no activity on this issue for two months. It will be closed in a week if no further activity occurs. If you would like to move this EIP forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions bot added the stale label Nov 20, 2021
@axic axic added discussions-to and removed stale labels Nov 29, 2021
@ethereum ethereum locked and limited conversation to collaborators Feb 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants