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

Dev eth v1.10.22 #64

Merged
merged 108 commits into from
Nov 2, 2023
Merged

Dev eth v1.10.22 #64

merged 108 commits into from
Nov 2, 2023

Conversation

cp-wjhan
Copy link

Apply ethereum v1.10.22

dbadoy and others added 30 commits September 22, 2023 09:24
…nc (ethereum#25210)

* eth/catalyst: disallow importing blocks via newPayload during snap sync

* eth/catalyst: make tests pass by using full sync only

* eth/catalysts: make the import delay a bit cleaner

* eth/catalyst: fix typo

Co-authored-by: Marius van der Wijden <[email protected]>

Co-authored-by: Marius van der Wijden <[email protected]>
Replace unnecessary fmt.Errorf with errors.New in accounts/abi/argument.go
* consensus/beacon: copy td value so we can modify it

* consensus/beacon: copy td value so we can modify it
…sts (ethereum#25119)

* core: don't validate state

* core: allow external validator

* core: revert

* core: comments

* Update blockchain_reader.go

* core: move SetValidator to blockchain.go

* core: rename method
* core: apply ttd override to genesis block

* core: apply overrides properly
…25105)

I have a EIP712 typehash using uint96, but it's currently not supported
by go-ethereum. This change fixes it.
* cmd/geth: add some missing argument count checks

* internal/flags: skip cmds with no action func in MigrateGlobalFlags

* internal/flags: add Merge

* cmd/devp2p: re-add listener config flags in discv4 commands
This change ensures the HTTP server will always terminate within
at most 5s, even when all connections are busy and do not become
idle.

Co-authored-by: Felix Lange <[email protected]>
Adds a native tracer that returns that in case of failure returns the error message or the revert reason of a transaction.

Co-authored-by: Martin Holst Swende <[email protected]>
…h local (ethereum#25157)

* internal/ethapi: error if tx args includes chain id that doesn't match local

* internal/ethapi: simplify code a bit

Co-authored-by: Péter Szilágyi <[email protected]>
-fixed conflicts : core/state_transition.go

During RPC calls such as eth_call and eth_estimateGas, st.evm.Config.NoBaseFee is set
which allows the gas price to be below the base fee. This results the tip being negative,
and balance being subtracted from the coinbase instead of added to it, which results in a
potentially negative coinbase balance interestingly. This can't happen during normal chain
processing as outside of RPC calls the gas price is required to be at least the base fee,
as NoBaseFee is false.

This change prevents this behavior by disabling fee payment when the fee is not set.

Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Felix Lange <[email protected]>
core, eth, les, trie: rework snap sync
rjl493456442 and others added 25 commits October 20, 2023 09:48
* all: polish tests

* core: apply feedback from Guillaume

* core: fix comment

* fix conflicts: core/genesis.go
* fix conflicts: core/state_processor.go
…m#25455)

signer/rules: register clef api properly when rules are used, fixes ethereum#25298
This adds support for building statically-linked executables using ci.go.

Static linking is enabled by default in Docker builds, making it possible to
use the geth executable in any Docker image, regardless of the Linux
distribution the Dockerfile is based on.

* fix conflicts: build/ci.go
Co-authored-by: Felix Lange <[email protected]>
core/genesis: calaverasAllocData no longer used
…#25528)

* params: set mainnet ttd to 58_750_000_000_000_000_000_000

* params: set mainnet ttd to 58_750_000_000_000_000_000_000
* core, trie: flush preimages to db on database close

Co-authored-by: rjl493456442 <[email protected]>

* rename Close to CommitPreimages for clarity

* core, trie: nitpick fixes

Co-authored-by: rjl493456442 <[email protected]>
Co-authored-by: Péter Szilágyi <[email protected]>
* core/state, trie, light: Add a DeleteAccount method

* review feedback

* Update database.go

* pr triage feedback

Co-authored-by: rjl493456442 <[email protected]>
* core: reduce system call about `os`

* avoid deprecated method
…5547)

typ will be nil when lookupStructType returns an error. cfg.Type should be used instead.
This avoids copying the input []byte while decoding trie nodes. In most
cases, particularly when the input slice is provided by the underlying
database, this optimization is safe to use.

For cases where the origin of the input slice is unclear, the copying version
is retained. The new code performs better even when the input must be
copied, because it is now only copied once in decodeNode.
* Fix some typos

* Fix some mistakes

* Revert 4byte.json

* Fix an incorrect fix

* Change files to fails

* Fix Conflicts: accounts/keystore/keystore_test.go,eth/state_accessor.go,les/fetcher.go
Unlock peerSet if there's an error in the downloader
* consensus/beacon: check ttd reached on pos blocks

* consensus/beacon: check ttd reached on pos blocks

* consensus/beacon: check ttd reached on pos blocks
This adds a cache for block logs which is shared by all filters. The cache
size of is configurable using the `--cache.blocklogs` flag.

Co-authored-by: Felix Lange <[email protected]>

* fix conflicts: cmd/geth/main.go,cmd/utils/flags.go
* accounts/abi: fix set function

* don't break things

* update test
…thereum#25569)

* eth/catalyst: warn less frequently if no beacon client is available

* eth/catalyst: tweak warning frequency a bit

* eth/catalyst: some more tweaks

* Update api.go

Co-authored-by: Felix Lange <[email protected]>
@cp-khs cp-khs requested review from sadoci and cp-jhjin November 1, 2023 00:32
Copy link

@sadoci sadoci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cp-khs cp-khs merged commit 21b2c75 into wemixarchive:dev Nov 2, 2023
5 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.