feat: filter logs for only the block being queried

CHANGELOG.md

@@ -44,6 +44,9 @@
 
 ### Fixed
 
+- [#5016](https://github.com/ChainSafe/forest/pull/5016) Filter logs for only
+  the block being queried.
+
 - [#4959](https://github.com/ChainSafe/forest/pull/4959) Re-enable garbage
   collection after implementing a "persistent" storage for manifests.
 

scripts/tests/api_compare/filter-list

@@ -14,5 +14,3 @@
 # TODO: https://github.com/ChainSafe/forest/issues/4701
 !Filecoin.EthGetTransactionByBlockHashAndIndex
 !Filecoin.EthGetTransactionByBlockNumberAndIndex
-# TODO: https://github.com/ChainSafe/forest/issues/5006
-!Filecoin.EthGetBlockReceipts

scripts/tests/api_compare/filter-list-offline

@@ -44,5 +44,3 @@
 # TODO: https://github.com/ChainSafe/forest/issues/4701
 !Filecoin.EthGetTransactionByBlockHashAndIndex
 !Filecoin.EthGetTransactionByBlockNumberAndIndex
-# TODO: https://github.com/ChainSafe/forest/issues/5006
-!Filecoin.EthGetBlockReceipts

src/rpc/methods/eth.rs

@@ -1154,6 +1154,17 @@ async fn new_eth_tx_receipt<DB: Blockstore + Send + Sync + 'static>(
 
     let mut events = vec![];
     EthEventHandler::collect_events(ctx, &parent_ts, None, &mut events).await?;
+
+    let current_block_hash = receipt.block_hash.clone();
+    events.retain(|event| {
+        if let Ok(block_hash) = event.tipset_key.cid() {
+            let event_block_hash: EthHash = block_hash.into();
+            event_block_hash == current_block_hash
+        } else {
+            false
+        }
+    });
+
     receipt.logs = eth_filter_logs_from_events(ctx, &events)?;
 
     Ok(receipt)