fix: CometBlockResultByNumber when height is 0 (#416)

* fix TendermintBlockResultByNumber when height is 0

Signed-off-by: Antonio Pitasi <[email protected]>

* add changelog entry

Signed-off-by: Antonio Pitasi <[email protected]>

---------

Signed-off-by: Antonio Pitasi <[email protected]>
Co-authored-by: Alex | Interchain Labs <[email protected]>

Author: Pitasi

CHANGELOG.md

@@ -10,6 +10,7 @@
 - [\#492](https://github.com/cosmos/evm/pull/492) Duplicate case switch to avoid empty execution block
 - [\#509](https://github.com/cosmos/evm/pull/509) Allow value with slashes when query token_pairs
 - [\#495](https://github.com/cosmos/evm/pull/495) Allow immediate SIGINT interrupt when mempool is not empty
+- [\#416](https://github.com/cosmos/evm/pull/416) Fix regression in CometBlockResultByNumber when height is 0 to use the latest block. This fixes eth_getFilterLogs RPC.
 - [\#545](https://github.com/cosmos/evm/pull/545) Check if mempool is not nil before accepting nonce gap error tx.
 
 ### IMPROVEMENTS

rpc/backend/blocks.go

@@ -203,6 +203,9 @@ func (b *Backend) CometHeaderByNumber(blockNum rpctypes.BlockNumber) (*cmtrpctyp
 // CometBlockResultByNumber returns a CometBFT-formatted block result
 // by block number
 func (b *Backend) CometBlockResultByNumber(height *int64) (*cmtrpctypes.ResultBlockResults, error) {
+	if height != nil && *height == 0 {
+		height = nil
+	}
 	res, err := b.RPCClient.BlockResults(b.Ctx, height)
 	if err != nil {
 		return nil, fmt.Errorf("failed to fetch block result from CometBFT %d: %w", *height, err)