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

Update EIP-7642: add earliestBlock and update receipts encoding #9237

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

fjl
Copy link
Contributor

@fjl fjl commented Jan 13, 2025

Here I am updating the eth/69 proposal with two changes:

  • Simplifying the receipts encoding. The original EIP proposed a new format for receipts with the bloom filter removed. It did that by mirroring the complex encoding for typed receipts which is also used by Ethereum consensus. Since the network protocol now introduces a deviation from the encoding used by consensus, there is no need to make it so complex.

    Ultimately all receipt types contain the same four fields. So with this update, I am changing the network encoding to be the same for all receipt types.

  • I am adding a new field earliestBlock in the Status message. This is somewhat similar to EIP-7542, which proposed adding a blockRange.

@fjl fjl requested a review from eth-bot as a code owner January 13, 2025 13:47
@github-actions github-actions bot added c-update Modifies an existing proposal s-stagnant This EIP is Stagnant t-networking labels Jan 13, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented Jan 13, 2025

File EIPS/eip-7642.md

Requires 1 more reviewers from @MariusVanDerWijden
Requires 1 more reviewers from @g11tech, @lightclient, @SamWilsn

@eth-bot eth-bot added the a-review Waiting on author to review label Jan 13, 2025
@eth-bot eth-bot changed the title EIP-7642: add earliestBlock and update receipts encoding Update EIP-7642: add earliestBlock and update receipts encoding Jan 13, 2025
title: eth/69 - Drop pre-merge fields
description: Drop unnecessary fields after the merge
title: eth/69 - history expiry and simpler receipts
description: Adds history serving window and removes bloom filter in receipt
author: Marius van der Wijden (@MariusVanDerWijden), Felix Lange <[email protected]>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
author: Marius van der Wijden (@MariusVanDerWijden), Felix Lange <[email protected]>
author: Marius van der Wijden (@MariusVanDerWijden), Felix Lange (@fjl)

this allows you to merge changes to the eip via the bot

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I know but I want to discuss a bit before merging.

Copy link
Contributor

@jflo jflo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besu team has no objections.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-review Waiting on author to review c-update Modifies an existing proposal s-stagnant This EIP is Stagnant t-networking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants