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

Release Candidate 2023.2.6 (dev -> main) #4479

Merged
merged 6 commits into from Aug 3, 2023
Merged

Release Candidate 2023.2.6 (dev -> main) #4479

merged 6 commits into from Aug 3, 2023

Conversation

sophoah
Copy link
Contributor

@sophoah sophoah commented Aug 3, 2023

Protocol changes :
#4475 - statedb: return correct validator code size, fix sync issue

Ops/Build changes
#4472 - Invalid dht prevents to start localnet
#4478 #4480 #4481 - upgrade docker in travis build to latest

Frozen and others added 3 commits July 22, 2023 11:38
Invalid dht prevents to start localnet.
The function `state_object.CodeSize` erroneously returns the validator
code size to be 0. This functionality is only used by the EVM, where the
size is overridden by the `ValidatorCodeFixEpoch` hard fork to be 0
anyway.

However, for nodes currently syncing blocks before the hard fork was
effective, this causes an error. A transaction which attempts to mint an
NFT to a validator address prior to the hard fork would have failed,
because `onERC1155Received` is not implemented by a validator's code.
With the erroneous line, the transaction goes through - causing
a node with the unpatched binary to reject the block.
* [build] upgrade docker in travis build to latest
* add comment to the before_install job

---------

Co-authored-by: Gheis Mohammadi <Gheis.Mohammadi@gmail.com>
MaxMustermann2 and others added 2 commits August 3, 2023 11:54
* [build] upgrade docker in travis using addons
* [build] upgrade docker in travis with debug command
* [build] retrying addons
* [build] debug log
* [build] clean debug log
@MaxMustermann2 MaxMustermann2 merged commit 53c2e66 into main Aug 3, 2023
3 checks passed
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.

None yet

6 participants