-
Notifications
You must be signed in to change notification settings - Fork 995
Change requests to use flat encoding #7771
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
Merged
Merged
Changes from 30 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
49a4411
rename requestsRoot to requestsHash
jframe cdc9bd5
generalised requests flat encoding and engine api changes
jframe cee6abc
javadoc
jframe 8f346f4
get tests passing
jframe 4e12780
get tests passing
jframe 42e6865
Merge remote-tracking branch 'upstream/main' into 7685_flat_encoding
jframe f0d5a24
clean code
jframe 0424317
change requests to single requestData for each requestType
jframe 86f5049
fix PoWBlockCreatorTest after requests data type change
jframe 722823e
don't return request type in getPayload result
jframe 24a33e7
include requests in t8n response
jframe 24de847
update contract addresses for consolidation requests and withdrawal r…
jframe 532cb9d
fix requestHash calculation
jframe 079bcfd
Ensure that execution requests always return a response
jframe eff46b0
revert changes to evm tool spec tests
jframe 1fdf0db
clean up
jframe 9c2a623
replace AbstractSystemCallRequestProcessor to concrete class and remo…
jframe afd9023
spotless
jframe 20ca19e
update evmtool tests for 7685 changes
jframe 3ee1b31
use empty requests hash prague fork at genesis
jframe 21b7aae
review suggestions
jframe 7574c84
engine API validation
jframe ac812b2
Merge remote-tracking branch 'upstream/main' into 7685_flat_encoding
jframe 95d1606
update plugin API hash
jframe 4c62cae
fix GenesisStateTest
jframe 985f4bf
update prague engine API ATs with updated contracts
jframe f298d46
update prague ATs using PragueAcceptanceTestHelper to use executionRe…
jframe 9114f4d
Merge branch 'main' into 7685_flat_encoding
jframe 3d4f2ec
Change BlockHeader isEmpty check to use the empty requests hash as pa…
jframe f1b412d
cleanup
jframe 8783ab7
pr suggestions
jframe 3833701
fix prague engine ATs after updating error message
jframe 2be3c0f
update SystemCallProcessorTest for case when contract doesn't exist
jframe dd72a27
review suggestions
jframe 64d7ed3
Merge branch 'main' into 7685_flat_encoding
jframe 157749a
Merge branch 'main' into 7685_flat_encoding
jframe 7b31ed0
Merge branch 'main' into 7685_flat_encoding
jframe 09a1cf7
Merge branch 'main' into 7685_flat_encoding
jframe 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 hidden or 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
12 changes: 7 additions & 5 deletions
12
acceptance-tests/tests/src/test/resources/dev/dev_prague.json
Large diffs are not rendered by default.
Oops, something went wrong.
23 changes: 14 additions & 9 deletions
23
acceptance-tests/tests/src/test/resources/jsonrpc/engine/prague/genesis.json
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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
File renamed without changes.
This file contains hidden or 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 hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.