chore: update go-ethereum

[ralph-pichler]

• update to go-ethereum v1.10.3 (last version which works with clef for non-london txs, see cannot sign non-eip 1559 with clef using external.ExternalSigner ethereum/go-ethereum#23273)
• migrate some swap keys to a readable format
• make hex encoding in keys explicit as string formatting might change (and add explicit tests for this)
• accept eip-2930 txs in handshake

This change is 

[mrekucci]

Reviewed 10 of 12 files at r1, 3 of 3 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @acud)

---

pkg/transaction/backendsimulation/backend.go, line 163 at r2 (raw file):

}

func (m *simulatedBackend) SuggestGasTipCap(ctx context.Context) (*big.Int, error) {

Since the ctx isn't used I'd suggest changing the name to _ in order to explicitly signal that.

[acud]

Reviewed 9 of 12 files at r1, 3 of 3 files at r2.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @ralph-pichler)

---

go.sum, line 794 at r2 (raw file):

github.com/mattn/go-runewidth v0.0.9 h1:Lm995f3rfxdpd6TSmuVCHVb/QhupuXlYr8sCI/QdE+0=
github.com/mattn/go-runewidth v0.0.9/go.mod h1:H031xJmbD/WCDINGzjvQ9THkh0rPKHF+m2gUSrubnMI=
github.com/mattn/go-sqlite3 v1.11.0/go.mod h1:FPy6KqzDD04eiIsT53CuJW3U88zkxoIYsOqkbpncsNc=

?? i wonder where is this coming from? did you go mod tidy?

---

pkg/statestore/leveldb/migration.go, line 138 at r2 (raw file):

			if err = s.Put(fixed, chequebookAddress); err != nil {
				return err
			}

what about s.Delete(key)?

[ralph-pichler]

Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @acud)

---

go.sum, line 794 at r2 (raw file):

Previously, acud (acud) wrote…

?? i wonder where is this coming from? did you go mod tidy?

yes. both go-ethereum and go-ens have this in their go.sum also, so probably one of their dependencies introduced this.

---

pkg/statestore/leveldb/migration.go, line 138 at r2 (raw file):

Previously, acud (acud) wrote…

what about s.Delete(key)?

added

---

pkg/transaction/backendsimulation/backend.go, line 163 at r2 (raw file):

Previously, mrekucci wrote…

Since the ctx isn't used I'd suggest changing the name to _ in order to explicitly signal that.

deleted as it was no longer needed after downgrade

[acud]

Reviewed 1 of 12 files at r1, 4 of 4 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ralph-pichler)