erigon2: eth_getBlockByHash returns incorrect miner field on matic

[quickchase]

System information

Erigon version: erigon/2.60.9/linux-amd64/go1.22.8

Chain/Network: bor-mainnet (matic)

Expected behaviour

eth_getBlockByHash output is the same as eth_getBlockByNumber output when the block/hashes are the same block

Actual behaviour

eth_getBlockByHash returns with the miner field populated, instead of 0x0000000000000000000000000000000000000000

Steps to reproduce the behaviour

# Set your node URL
NODE=http://localhost:8545

# Get the block hash of 0x3d146d3
curl -s -H 'Content-Type: application/json' $NODE -d '{"jsonrpc":"2.0","id":1,"method":"eth_getBlockByNumber","params":["0x3d146d3",false]}' | jq '.result.hash'
"0x8d7953e6bc4c154e1b4561554effa23f17850829bdfb755d7ff6aa2b30232dc4"

# Check the miner field of 0x3d146d3
curl -s -H 'Content-Type: application/json' $NODE -d '{"jsonrpc":"2.0","id":1,"method":"eth_getBlockByNumber","params":["0x3d146d3",false]}' | jq '.result.miner'
"0x0000000000000000000000000000000000000000"

# Check the miner field of 0x3d146d3, but this time using the hash
curl -s -H 'Content-Type: application/json' $NODE -d '{"jsonrpc":"2.0","id":1,"method":"eth_getBlockByHash","params":["0x8d7953e6bc4c154e1b4561554effa23f17850829bdfb755d7ff6aa2b30232dc4",false]}' | jq '.result.miner'
"0x794e44d1334a56fea7f4df12633b88820d0c5888"

As you can see, the miner field is wrong for eth_getBlockByHash

[cshintov]

Happening for us as well.. for all blocks..

Fetching the block hash for block number 0x3dc03b1...
Block Hash: 0x1e3b36405cac88d4980cf5e3893f8b773a9833fee87f7e33253a773d1a08f3ed
Checking the miner field for block number 0x3dc03b1...
Miner field for block 0x3dc03b1: 0x0000000000000000000000000000000000000000
Checking the miner field for the block using its hash 0x1e3b36405cac88d4980cf5e3893f8b773a9833fee87f7e33253a773d1a08f3ed...
Miner field for block hash 0x1e3b36405cac88d4980cf5e3893f8b773a9833fee87f7e33253a773d1a08f3ed: 0xfcccd43296d9c1601a904eca9b339d94a5e5e098

[cshintov]

Seems like block by hash has special handling for miner field..

erigon/turbo/jsonrpc/eth_block.go

Lines 322 to 325 in 0bff9b8

	if chainConfig.Bor != nil {
	borConfig := chainConfig.Bor.(*borcfg.BorConfig)
	response["miner"], _ = ecrecover(block.Header(), borConfig)
	}