chore_: remove CallBlockHashByTransaction workaround for bad block hash calculation

Author: dlipicar

rpc/chain/client.go

@@ -783,24 +783,6 @@ func (c *ClientWithFallback) GetBaseFeeFromBlock(ctx context.Context, blockNumbe
 	return baseGasFee, err
 }
 
-func (c *ClientWithFallback) CallBlockHashByTransaction(ctx context.Context, blockNumber *big.Int, index uint) (common.Hash, error) {
-	rpcstats.CountCallWithTag("eth_FullTransactionByBlockNumberAndIndex", c.tag)
-
-	res, err := c.makeCall(
-		ctx, c.ethClients, func(client ethclient.RPSLimitedEthClientInterface) (interface{}, error) {
-			return client.CallBlockHashByTransaction(ctx, blockNumber, index)
-		},
-	)
-
-	c.toggleConnectionState(err)
-
-	if err != nil {
-		return common.HexToHash(""), err
-	}
-
-	return res.(common.Hash), nil
-}
-
 func (c *ClientWithFallback) GetWalletNotifier() func(chainId uint64, message string) {
 	return c.WalletNotifier
 }

rpc/chain/ethclient/eth_client.go

@@ -62,7 +62,6 @@ type EthClientInterface interface {
 	GetBaseFeeFromBlock(ctx context.Context, blockNumber *big.Int) (string, error)
 	bind.ContractCaller
 	bind.ContractBackend
-	CallBlockHashByTransaction(ctx context.Context, blockNumber *big.Int, index uint) (common.Hash, error)
 }
 
 // EthClient implements EthClientInterface
@@ -82,13 +81,6 @@ func (ec *EthClient) BatchCallContext(ctx context.Context, b []rpc.BatchElem) er
 	return ec.rpcClient.BatchCallContext(ctx, b)
 }
 
-// go-ethereum's `Transaction` items drop the blkHash obtained during the RPC call.
-// This function preserves the additional data. This is the cheapest way to obtain
-// the block hash for a given block number.
-func (ec *EthClient) CallBlockHashByTransaction(ctx context.Context, blockNumber *big.Int, index uint) (common.Hash, error) {
-	return callBlockHashByTransaction(ctx, ec.rpcClient, blockNumber, index)
-}
-
 func (ec *EthClient) GetBaseFeeFromBlock(ctx context.Context, blockNumber *big.Int) (string, error) {
 	feeHistory, err := ec.FeeHistory(ctx, 1, blockNumber, nil)
 

rpc/chain/ethclient/rpc.go

@@ -15,7 +15,6 @@ type Reader interface {
 	BalanceAt(ctx context.Context, account common.Address, blockNumber *big.Int) (*big.Int, error)
 	NonceAt(ctx context.Context, account common.Address, blockNumber *big.Int) (uint64, error)
 	HeaderByNumber(ctx context.Context, number *big.Int) (*types.Header, error)
-	CallBlockHashByTransaction(ctx context.Context, blockNumber *big.Int, index uint) (common.Hash, error)
 	NetworkID() uint64
 }
 

services/wallet/balance/balance_cache.go

@@ -305,14 +305,6 @@ func (tc *TestClient) HeaderByNumber(ctx context.Context, number *big.Int) (*typ
 	return header, nil
 }
 
-func (tc *TestClient) CallBlockHashByTransaction(ctx context.Context, blockNumber *big.Int, index uint) (common.Hash, error) {
-	err := tc.countAndlog("CallBlockHashByTransaction")
-	if err != nil {
-		return common.Hash{}, err
-	}
-	return common.BigToHash(blockNumber), nil
-}
-
 func (tc *TestClient) GetBaseFeeFromBlock(ctx context.Context, blockNumber *big.Int) (string, error) {
 	err := tc.countAndlog("GetBaseFeeFromBlock")
 	return "", err

services/wallet/transfer/commands_sequential_test.go

@@ -177,17 +177,11 @@ func checkRangesWithStartBlock(parent context.Context, client balance.Reader, ca
 				}
 			}
 			if new(big.Int).Sub(to, from).Cmp(one) == 0 {
-				// WARNING: Block hash calculation from plain header returns a wrong value.
 				header, err := client.HeaderByNumber(ctx, to)
 				if err != nil {
 					return err
 				}
-				// Obtain block hash from first transaction
-				blockHash, err := client.CallBlockHashByTransaction(ctx, to, 0)
-				if err != nil {
-					return err
-				}
-				c.PushHeader(toDBHeader(header, blockHash, account))
+				c.PushHeader(toDBHeader(header, account))
 				return nil
 			}
 			mid := new(big.Int).Add(from, to)

services/wallet/transfer/concurrent.go

@@ -81,10 +81,6 @@ func (f balancesFixture) NetworkID() uint64 {
 	return 0
 }
 
-func (f balancesFixture) CallBlockHashByTransaction(ctx context.Context, blockNumber *big.Int, index uint) (common.Hash, error) {
-	return common.HexToHash("0x0"), nil
-}
-
 type batchesFixture [][]Transfer
 
 func (f batchesFixture) GetTransfersByNumber(ctx context.Context, number *big.Int) (rst []Transfer, err error) {

services/wallet/transfer/concurrent_test.go

@@ -33,9 +33,9 @@ type DBHeader struct {
 	Loaded bool
 }
 
-func toDBHeader(header *types.Header, blockHash common.Hash, account common.Address) *DBHeader {
+func toDBHeader(header *types.Header, account common.Address) *DBHeader {
 	return &DBHeader{
-		Hash:      blockHash,
+		Hash:      header.Hash(),
 		Number:    header.Number,
 		Timestamp: header.Time,
 		Loaded:    false,