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

Bump github.com/ethereum/go-ethereum from 1.9.25 to 1.10.9 #232

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 4, 2021

Bumps github.com/ethereum/go-ethereum from 1.9.25 to 1.10.9.

Release notes

Sourced from github.com/ethereum/go-ethereum's releases.

Attican Beta (v1.10.9)

Geth v1.10.9 is a maintenance release containing mostly bug fixes.

Chain tracing has received quite a bit of attention during this release cycle. JS-based tracing now supports additional callbacks for entry and exit of contract calls, improving performance if processing individual opcodes is not needed.

Geth command changes

  • The 'evm' command has a new subcommand for testing tx decoding. (#23494)
  • 'evm t8n' now calculates and returns block difficulty. (#23353, #23507)
  • Legacy flags --rpc, --rpcaddr, --rpcport, etc. are no longer supported. (#23358)
  • Legacy debugging flags --pprofport, --pprofaddr, etc. are also no longer supported. (#23368)
  • When initializing Clique-based private networks, zero-length extradata in genesis.json now prints an error message instead of crashing. (#23538)

Go library changes

  • Contract bindings created by accounts/abi/bind now validate log event signatures. This prevents accidentally decoding events with the wrong signature. (#23230)
  • A crash in accounts/abi when decoding struct-typed Solidity return values is resolved. (#23573)
  • Writes to the ancient database are now batched internally for improved performance. (#23462)
  • Multiple data races in transaction pool code are fixed. (#23474)
  • Comprehensive benchmarks for RLP encoding/decoding of consensus types have been added. (#23190)
  • RLP encoding of slices and arrays is slightly faster. (#23467)
  • rpc.BlockNumber now implements encoding.TextMarshaler. (#23324)
  • The Account type has been moved from package core/state to core/types. (#23567)
  • For crypto/cloudflare/bn256 EC curve, in-place addition and unmarshalling now works correctly. (#23419)
  • A very rare crash in the background 'bloombits' indexer is resolved. (#23437)

RPC/GraphQL changes

  • JS tracing of EVM execution now provides additional callbacks for call entry/exit. Using these callbacks instead of 'step' can yield a 10-100x tracing speedup if you don't need to process every VM opcode. (#23087)
  • The '4byte' built-in tracer now uses enter/exit. (#23622)
  • A state database corruption bug caused by tracer re-execution of old blocks is resolved. (#23632)
  • The new debug_intermediateRoots method computes per-transaction state roots of a block. (#23594)
  • EVM memory and return data are no longer captured by default when tracing. (#23558)
  • EVM execution is now aborted on the server side when tracing is interrupted. (#23580)
  • Broken WebSocket connections are now detected better and their subscriptions report an error instead of hanging indefinitely. (#23556)
  • personal_sendTransaction now supports both "input" and "data" arguments, just like eth_sendTransaction. (#23476)
  • Log filtering performance is improved. (#23147)
  • Transaction access lists returned by GraphQL are now correct. (#23650)
  • The debug_stacks method now supports an optional filter expression. (#23605)
  • For clique blocks returned by RPC, the "miner" field once again contains the actual block coinbase field instead of the derived block signer. This fixes a regression where clients would no longer be able to verify the block seal signature. (#23466)

Networking

  • The eth/65 peer-to-peer protocol is no longer supported. Geth only supports eth/66 as of this release. (#23456)
  • The cross-client eth protocol tests suite better distinguishes eth/65 and eth/66. (#23568)
  • ENR sequence numbers are now initialized as a timestamp. This prevents issues when the p2p nodes database is dropped/re-created while keeping the nodekey the same. (#19903)
  • Several data races are resolved in packages p2p and p2p/enode. (#23434)
  • Note: to simplify the ongoing rewrite of eth/downloader, the package has been duplicated temporarily. The additional copy will be removed later. (#23561)

Build

... (truncated)

Commits
  • eae3b19 params: release go-ethereum v1.10.9 stable
  • 3a6fe69 eth/protocols/snap, trie: better error-handling (#23657)
  • 42bc194 graphql: add storage slots to access list (#23650)
  • a541fbe eth/protocols/eth: simplify peer known block/txs caches (#23649)
  • 3531ca2 eth/tracers: avoid unsyncronized mutations on trie database (#23632)
  • 92c5d10 accounts/abi/bind: check event signature before parsing (#23230)
  • 783e97e core/rawdb: avoid unnecessary receipt processing for log filtering (#23147)
  • ab2caae eth/tracers: implement debug.intermediateRoots (#23594)
  • 443afc9 core/state: move state account to core/types + abstracted "write account to t...
  • ac7baea eth/tracers: re-write of 4byte tracer using enter/exit (#23622)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps [github.com/ethereum/go-ethereum](https://github.com/ethereum/go-ethereum) from 1.9.25 to 1.10.9.
- [Release notes](https://github.com/ethereum/go-ethereum/releases)
- [Commits](ethereum/go-ethereum@v1.9.25...v1.10.9)

---
updated-dependencies:
- dependency-name: github.com/ethereum/go-ethereum
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file backend Pull requests that update Go code labels Nov 4, 2021
@loleg
Copy link
Contributor

loleg commented Dec 2, 2021

Currently failing due to the main/handlers/blockchain test with this error:

# github.com/ProxeusApp/proxeus-core/main/handlers/blockchain/ethglue [github.com/ProxeusApp/proxeus-core/main/handlers/blockchain/ethglue.test]
main/handlers/blockchain/ethglue/connection_test.go:24:11: cannot use &ethClient (type *FakeETHClient) as type ETHClientIF in return argument:
	*FakeETHClient does not implement ETHClientIF (missing SuggestGasTipCap method)
main/handlers/blockchain/ethglue/connection_test.go:36:11: cannot use &ethClient (type *FakeETHClient) as type ETHClientIF in return argument:
	*FakeETHClient does not implement ETHClientIF (missing SuggestGasTipCap method)
main/handlers/blockchain/ethglue/connection_test.go:50:11: cannot use &ethClient (type *FakeETHClient) as type ETHClientIF in return argument:
	*FakeETHClient does not implement ETHClientIF (missing SuggestGasTipCap method)
main/handlers/blockchain/ethglue/nonce_test.go:59:27: cannot use pendingNonceMock{} (type pendingNonceMock) as type ETHClientIF in argument to m.OnDial:
	pendingNonceMock does not implement ETHClientIF (missing SuggestGasTipCap method)
# github.com/ProxeusApp/proxeus-core/main/handlers/blockchain [github.com/ProxeusApp/proxeus-core/main/handlers/blockchain.test]
main/handlers/blockchain/log_subscriber_test.go:37:4: cannot use ethClient (type *ethglue.FakeETHClient) as type ethglue.ETHClientIF in return argument:
	*ethglue.FakeETHClient does not implement ethglue.ETHClientIF (missing SuggestGasTipCap method)
main/handlers/blockchain/log_subscriber_test.go:69:4: cannot use ethClient (type *ethglue.FakeETHClient) as type ethglue.ETHClientIF in return argument:
	*ethglue.FakeETHClient does not implement ethglue.ETHClientIF (missing SuggestGasTipCap method)
main/handlers/blockchain/log_subscriber_test.go:85:4: cannot use ethClient (type *ethglue.FakeETHClient) as type ethglue.ETHClientIF in return argument:
	*ethglue.FakeETHClient does not implement ethglue.ETHClientIF (missing SuggestGasTipCap method)

@loleg loleg requested a review from m-javani December 2, 2021 09:41
@loleg
Copy link
Contributor

loleg commented Dec 8, 2021

@Javany could you please look at the issue with this upgrade.

@m-javani
Copy link
Contributor

m-javani commented Dec 9, 2021

since go-ethereum has changed, we need the following updates:

  • in main/handlers/blockchain/ethglue/interfaces.go
    • at line 22, we need to add the following line to the ETHClientIF interface:
     SuggestGasTipCap(ctx context.Context) (*big.Int, error)
    
    • and add the following method to the end of the file:
         func (me *FakeETHClient) SuggestGasTipCap(ctx context.Context) (*big.Int, error) {
                panic("implement me")
         }
      
  • in the main/handlers/blockchain/ethglue/nonce_test.go
    • at line 56, we need to add this method:
        func (_ pendingNonceMock) SuggestGasTipCap(ctx context.Context) (*big.Int, error) {
              panic("implement me")
        }
    

Copy link
Contributor

@m-javani m-javani left a comment

Choose a reason for hiding this comment

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

needs updating. left a comment

@loleg
Copy link
Contributor

loleg commented Dec 9, 2021

Thanks. If you can push those changes to the dependabot/go_modules/github.com/ethereum/go-ethereum-1.10.9 branch that would be ideal.

Also I get this error when upgrading, even after clearing my cache:

go get all: module github.com/ethereum/go-ethereum@upgrade found (v1.10.13), but does not contain package github.com/ethereum/go-ethereum/p2p/discv5

We may as well move to the latest v1.10 release, no?

@m-javani
Copy link
Contributor

m-javani commented Dec 9, 2021

pushed the changes and the bug relating to FakeETHClient is resolved.
now, it fails on building sass

@loleg
Copy link
Contributor

loleg commented Dec 10, 2021 via email

@loleg loleg self-requested a review December 10, 2021 17:10
@loleg loleg mentioned this pull request Dec 11, 2021
@loleg loleg merged commit 3a29a6a into master Dec 13, 2021
@loleg loleg deleted the dependabot/go_modules/github.com/ethereum/go-ethereum-1.10.9 branch December 13, 2021 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Pull requests that update Go code dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants