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

Merge v1.11.4 #221

Merged
merged 18 commits into from
Jun 7, 2023
Merged

Merge v1.11.4 #221

merged 18 commits into from
Jun 7, 2023

Conversation

Tristan-Wilson
Copy link
Member

@Tristan-Wilson Tristan-Wilson commented May 19, 2023

Resolved conflicts related to core.Message being changed from an interface to a struct. See https://github.com/ethereum/go-ethereum/releases/tag/v1.11.4

Testing done

Same set of tests is failing as for v1.11.3 #218

$ grep FAIL failing_tests_v1.11.4.txt 
--- FAIL: TestSequentialRead (0.00s)
--- FAIL: TestSequentialReadByteLimit (0.00s)
--- FAIL: TestFreezerReadonly (0.00s)
--- FAIL: TestRandom (0.00s)
FAIL    github.com/ethereum/go-ethereum/core/rawdb      0.020s
--- FAIL: TestPostCapBasicDataAccess (0.00s)
FAIL
FAIL    github.com/ethereum/go-ethereum/core/state/snapshot     3.577s
--- FAIL: TestCallTracerNative (0.00s)
    --- FAIL: TestCallTracerNative/simple (0.00s)
    --- FAIL: TestCallTracerNative/revert (0.00s)
    --- FAIL: TestCallTracerNative/create (0.00s)
    --- FAIL: TestCallTracerNative/revertReason (0.00s)
    --- FAIL: TestCallTracerNative/oog (0.00s)
    --- FAIL: TestCallTracerNative/innerInstafail (0.00s)
    --- FAIL: TestCallTracerNative/delegatecall (0.00s)
    --- FAIL: TestCallTracerNative/innerThrowOuterRevert (0.00s)
    --- FAIL: TestCallTracerNative/simpleOnlytop (0.00s)
    --- FAIL: TestCallTracerNative/selfdestruct (0.00s)
    --- FAIL: TestCallTracerNative/innerRevertReason (0.00s)
    --- FAIL: TestCallTracerNative/throw (0.00s)
    --- FAIL: TestCallTracerNative/innerCreateOogOuterThrow (0.00s)
    --- FAIL: TestCallTracerNative/deepCalls (0.00s)
--- FAIL: TestCallTracerNativeWithLog (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/simple (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/calldata (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/withOnlyTopCall (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/txPartialFailed (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/notopic (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/txFailed (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/delegatecall (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/multilogs (0.00s)
    --- FAIL: TestCallTracerNativeWithLog/multiContracts (0.01s)
--- FAIL: TestZeroValueToNotExitCall (0.00s)
FAIL    github.com/ethereum/go-ethereum/eth/tracers/internal/tracetest  0.081s
FAIL

fjl and others added 15 commits March 7, 2023 18:18
…gapped (#26719)

This change prints out more information about the problem, in the case where geth detects a gap between leveldb and ancients, so we can determine more exactly where the gap is (what the first missing is). Also prints out more metadata. 

---------

Co-authored-by: Martin Holst Swende <martin@swende.se>
…(#26770)

Accept all primitive types in Solidity for EIP-712 from intN, uintN, intN[], uintN[] for N as 0 to 256 in multiples of 8

---------

Co-authored-by: Martin Holst Swende <martin@swende.se>
reverts #26021, to use the upstream bigint instead.
This changes the test to match the comment description. Using timestampedConfig in this test case is incorrect, the comment says 'local is at Gray Glacier' and isn't aware of more forks.
Here, the core.Message interface turns into a plain struct and
types.Message gets removed.

This is a breaking change to packages core and core/types. While we do
not promise API stability for package core, we do for core/types. An
exception can be made for types.Message, since it doesn't have any
purpose apart from invoking the state transition in package core.
types.Message was also marked deprecated by the same commit it
got added in, 4dca5d4 (November 2016).

The core.Message interface was added in December 2014, in commit
db49417, for the purpose of 'testing' state transitions. It's the
same change that made transaction struct fields private. Before that,
the state transition used *types.Transaction directly.

Over time, multiple implementations of the interface accrued across
different packages, since constructing a Message is required whenever
one wants to invoke the state transition. These implementations all
looked very similar, a struct with private fields exposing the fields
as accessor methods.

By changing Message into a struct with public fields we can remove all
these useless interface implementations. It will also hopefully
simplify future changes to the type with less updates to apply across
all of go-ethereum when a field is added to Message.

---------

Co-authored-by: Felix Lange <fjl@twurst.com>
Since forks are now scheduled by block time, it can be necessary
to check the timestamp of a block while generating transactions.
This adds two new rules to the transaction pool:

- A future transaction can not evict a pending transaction.
- A transaction can not overspend available funds of a sender.

---

Co-authored-by: dwn1998 <42262393+dwn1998@users.noreply.github.com>
Co-authored-by: Martin Holst Swende <martin@swende.se>
Resolved conflicts related to core.Message being changed from interface
to struct.
@Tristan-Wilson Tristan-Wilson changed the base branch from master to merge-v1.11.3 May 19, 2023 17:39
This previously existed as b7b9359 but it got removed accidentally
during the v1.11.4 merge.
@Tristan-Wilson Tristan-Wilson marked this pull request as ready for review May 20, 2023 02:40
Base automatically changed from merge-v1.11.3 to master June 6, 2023 05:31
Copy link
Collaborator

@PlasmaPower PlasmaPower left a comment

Choose a reason for hiding this comment

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

LGTM

@PlasmaPower PlasmaPower merged commit a2132df into master Jun 7, 2023
@PlasmaPower PlasmaPower deleted the merge-v1.11.4 branch June 7, 2023 06:27
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.