Skip to content

Feature/eip 4844 v6.1#5784

Merged
flcl42 merged 4 commits intofeature/eip-4844-v6-batchfrom
feature/eip-4844-v6.1
Jun 9, 2023
Merged

Feature/eip 4844 v6.1#5784
flcl42 merged 4 commits intofeature/eip-4844-v6-batchfrom
feature/eip-4844-v6.1

Conversation

@LukaszRozmej
Copy link
Member

No description provided.

@LukaszRozmej LukaszRozmej requested a review from flcl42 June 7, 2023 10:36
@LukaszRozmej LukaszRozmej changed the base branch from feature/eip-4844-v6 to feature/eip-4844-v6-batch June 7, 2023 10:56
@flcl42 flcl42 force-pushed the feature/eip-4844-v6.1 branch from 497cab6 to bf62734 Compare June 7, 2023 13:28
@flcl42 flcl42 merged commit 6e3ac40 into feature/eip-4844-v6-batch Jun 9, 2023
@flcl42 flcl42 deleted the feature/eip-4844-v6.1 branch June 9, 2023 07:48
LukaszRozmej added a commit that referenced this pull request Jun 12, 2023
* Workaround for reorgs

* Change DATAHASH to BLOBHASH

* Add blob hashes to newpayloadv3
(CL can't verify them in RLP, sends to EL)

* `To` should not be empty for blobs

* Fix tx type detection

* Add non-zero blobs test case to strengthen blob verification test coverage

* Feature/eip 4844 v6.1 (#5784)

* Added ExecutionPayload._transactions to avoid multiple RLP decoding

* Simplify and refactor validation

* Add test

* Uglify to comply

---------

Co-authored-by: Alexey Osipov <me@flcl.me>

---------

Co-authored-by: Lukasz Rozmej <lukasz.rozmej@gmail.com>
jmederosalvarado pushed a commit that referenced this pull request Jun 13, 2023
* Workaround for reorgs

* Change DATAHASH to BLOBHASH

* Add blob hashes to newpayloadv3
(CL can't verify them in RLP, sends to EL)

* `To` should not be empty for blobs

* Fix tx type detection

* Add non-zero blobs test case to strengthen blob verification test coverage

* Feature/eip 4844 v6.1 (#5784)

* Added ExecutionPayload._transactions to avoid multiple RLP decoding

* Simplify and refactor validation

* Add test

* Uglify to comply

---------

Co-authored-by: Alexey Osipov <me@flcl.me>

---------

Co-authored-by: Lukasz Rozmej <lukasz.rozmej@gmail.com>
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

Successfully merging this pull request may close these issues.

2 participants

Comments