reverted historical proofs

fluentlabs-xyz · Nov 8, 2023 · 90b313f · 90b313f
1 parent caf5bee
commit 90b313f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/crates/rpc/rpc/src/eth/api/state.rs b/crates/rpc/rpc/src/eth/api/state.rs
@@ -103,9 +103,9 @@ where
         };
 
         // TODO: remove when HistoricalStateProviderRef::proof is implemented
-        // if !is_latest_block {
-        //     return Err(EthApiError::InvalidBlockRange)
-        // }
+        if !is_latest_block {
+            return Err(EthApiError::InvalidBlockRange)
+        }
 
         let this = self.clone();
         self.inner

diff --git a/crates/storage/provider/src/providers/state/historical.rs b/crates/storage/provider/src/providers/state/historical.rs
@@ -10,7 +10,7 @@ use reth_db::{
     transaction::DbTx,
     BlockNumberList,
 };
-use reth_interfaces::{RethError, RethResult};
+use reth_interfaces::RethResult;
 use reth_primitives::{
     trie::AccountProof, Account, Address, BlockNumber, Bytecode, StorageKey, StorageValue, B256,
 };
@@ -240,10 +240,8 @@ impl<'b, TX: DbTx> StateProvider for HistoricalStateProviderRef<'b, TX> {
     }
 
     /// Get account and storage proofs.
-    fn proof(&self, address: Address, keys: &[B256]) -> RethResult<AccountProof> {
-        let proof = reth_trie::proof::Proof::new(self.tx);
-        proof.account_proof(address, keys)
-            .map_err(|_| RethError::Custom("proof generation failed".to_string()))
+    fn proof(&self, _address: Address, _keys: &[B256]) -> RethResult<AccountProof> {
+        Err(ProviderError::StateRootNotAvailableForHistoricalBlock.into())
     }
 }