-
Notifications
You must be signed in to change notification settings - Fork 759
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
pectra devnet4: implement pectra devnet4 spec #3706
Merged
Merged
Changes from all commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
bcb48d2
vm: update the system contract addresses for prague devnet4
g11tech 1c30c2a
change the requests root from trie root to flat root and update examp…
g11tech 73003e7
convert requests to flat type across util,block and vm
g11tech 7cd91a2
bundle execution requests separately from execution payload
g11tech b4c704b
Merge branch 'master' into devnet4-contracts
jochem-brouwer 420a7aa
t8ntool: update to devnet-4 interface support
jochem-brouwer 1b65e0c
update devnet-4 to EIP PRs 8924, 8394
jochem-brouwer 6d39f31
t8ntool hotfix to fix state tests
jochem-brouwer 7d023c4
refactor cl requests to the new simplified version
g11tech 2998547
remove requests from the block and modify associated code paths
g11tech 5d2ede2
remove storing and retriving of requests from blockchain
g11tech 5fe681b
modify the deposit, withdrawal and consolidation requests accumulatio…
g11tech 613c900
modfiy the 7002 eip spec along with the new contract and debug and fi…
g11tech 9b5caf2
modify code to correctly patch generated requests on getpayload/build…
g11tech a62ed49
fix the newpayload engine codeflow to validate the cl requests
g11tech 7e898d4
remove the requests from eth rpc and blockfetcher p2p
g11tech a5aa2db
modify debug and fix 6110 deposit spec test
g11tech 3eafb13
update the vm 7685 spec and add todos for later consideration
g11tech 638acc3
fix t8ntool rq output
jochem-brouwer 965eb86
vm: fix 6110 requests
jochem-brouwer 2fea7ae
update request to just store bytes and expose getters for data and ty…
g11tech 8b641de
repo: rename requestsRoot -> requestsHash
jochem-brouwer dae997e
vm: fix import (fix docker build)
jochem-brouwer bef2f34
client: correctly return request data (not including type)
jochem-brouwer 860c621
fix the ingress, generation and propagation of execution requests/req…
g11tech a6dde76
Update 6110 example
scorbajio e7ba8b9
Use sha256 constant for default
scorbajio ee9a33d
Fix asserts
scorbajio 239a63d
Add sha256 empty string constant
scorbajio 4f9ae46
Update block REAME examples
scorbajio f0b5acc
Reuse already computed hash
scorbajio d7a33f8
Merge branch 'master' of github.com:ethereumjs/ethereumjs-monorepo in…
scorbajio 676b93b
Fix buildBlock tests
scorbajio b1bae17
Fix vm api tests
scorbajio 9d3652f
Fix client tests
scorbajio ea935e7
Fix tests
scorbajio 7e1c382
packages: add requests hash to genesis block (#3771)
spencer-tb 9d5822f
util: correctly report empty rq hash
jochem-brouwer dc93f8f
blockchain/util: remove sha256_empty_rh from exported util constants
jochem-brouwer 9c64353
Merge branch 'master' into devnet4-contracts
scorbajio 328cd53
Remove requests from being passed in as blockData
scorbajio c7f4640
Remove old tests that do not conform to new devnet4 specs
scorbajio a973b43
Remove old test that does not conform to new devnet4 specs
scorbajio 000839d
make linter happy
jochem-brouwer d91b666
block: make tsc happy
jochem-brouwer 84f40e3
block: remove obsolete examples
jochem-brouwer d3e6cca
util: make tsc happy
jochem-brouwer 802145d
vm: make linter and tsc happy
jochem-brouwer 88eb200
blockchain: make linter happy
jochem-brouwer adff539
vm: fix example
jochem-brouwer 46ec57d
make cspell happy
jochem-brouwer 5446fb8
client/util/vm: simplify CLRequest
jochem-brouwer 152ae16
Merge branch 'master' into devnet4-contracts
acolytec3 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,6 +12,7 @@ | |
} | ||
], | ||
"words": [ | ||
"EEST", | ||
"paulmillr", | ||
"t8ntool", | ||
"!Json", | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs update, now uses
sha256
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the example and will push updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The 7002 example in the block package fails still, but since ssz isn't supported, we would have to provide a very minimal example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The SSZ support is a misunderstanding: we do not need to import SSZ. The contract (which generates those requests) internally LOGs the data SSZ-encoded (we can just read the log as opaque bytes and use that). We should update the example (I'll take a look what it does currently)