celo-org / celo-blockchain

Official repository for the golang Celo Blockchain
https://celo.org
GNU Lesser General Public License v3.0
552 stars 196 forks source link

eth_getBlockReceipts is broken, because it does not take into account the celo block receipt #2293

Closed piersy closed 3 months ago

piersy commented 3 months ago

Expected Behavior

Getting the block receipts with cast should work

Actual Behavior

cast rpc eth_getBlockReceipts `cast to-hex 53906`
Error:
(code: -32000, message: receipts length mismatch: 0 vs 1, data: None)

Solution: Ensure we handle the following edge case and any others in this function taking into account the celo block receipt: https://github.com/celo-org/celo-blockchain/blob/6351612d67d4cf5e18708a0536f0cb43afa19e43/internal/ethapi/api.go#L840-L842

System Information

Commit: 6351612d67d4cf5e18708a0536f0cb43afa19e43