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.

Sign up for GitHub

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

Jump to bottom

Receipt: add missed fields #13047

Open
AskAlexSharov opened this issue Dec 9, 2024 · 0 comments
Open

Receipt: add missed fields #13047

AskAlexSharov opened this issue Dec 9, 2024 · 0 comments
Labels
erigon3 imp2 Medium importance
Milestone
3.0.0-beta2

Comments

@AskAlexSharov
Copy link
Collaborator

Add next fields (and maybe logic related to them from geth):

EffectiveGasPrice *big.Int          `json:"effectiveGasPrice"` // required, but tag omitted for backwards compatibility
BlobGasUsed       uint64            `json:"blobGasUsed,omitempty"
BlobGasPrice      *big.Int          `json:"blobGasPrice,omitempty"`

do we store enough data in ReceiptDomain?

receiptMarshaling also need add:

EffectiveGasPrice *hexutil.Big
BlobGasUsed       hexutil.Uint64
BlobGasPrice      *hexutil.Big

run make gencodec
@AskAlexSharov AskAlexSharov added the imp2 Medium importance label Dec 9, 2024
@AskAlexSharov AskAlexSharov added this to the 3.0.0-beta2 milestone Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
erigon3 imp2 Medium importance
Projects
None yet
Milestone
3.0.0-beta2
Development

No branches or pull requests
1 participant
@AskAlexSharov