Add support for eth_getAccount

[mattsse]

Describe the feature

ref https://www.quicknode.com/docs/ethereum/eth_getAccount

TODO

• add endpoint to trait, like

  reth/crates/rpc/rpc-eth-api/src/core.rs

  Lines 55 to 58 in 390f30a

  	/// Returns information about a block by hash.
  	#[method(name = "getBlockByHash")]
  	async fn block_by_hash(&self, hash: B256, full: bool) -> RpcResult<Option<RichBlock>>;

• implement it like

  reth/crates/rpc/rpc-eth-api/src/core.rs

  Lines 372 to 376 in 390f30a

  	/// Handler for: `eth_getBlockByHash`
  	async fn block_by_hash(&self, hash: B256, full: bool) -> RpcResult<Option<RichBlock>> {
  	trace!(target: "rpc::eth", ?hash, ?full, "Serving eth_getBlockByHash");
  	Ok(EthBlocks::rpc_block(self, hash.into(), full).await?)
  	}

the latter likely needs more work, so this can be two prs where the first just introduces the trait function and when figure out how to get the account's storage root cc @rkrasiuk

Additional context

No response

[mvares]

[mvares]

   graph TD
    A[Get Account]
    B[codeHash]
    C[storageRoot]
    D[balance]
    E[nonce]

    A --> B
    A --> C
    A --> D
    A --> E

Loading

As was done here, this can be a starting point to address this the second question of this issue.

[mattsse]

@rkrasiuk I assume getting the storageroot at any block number isn't that easy, same issue as eth_getProof?

[rkrasiuk]

@mattsse actually it's much easier since you only need to load storage changes for a given account. so, as long as all touched storage can fit in memory, we would be good

[mattsse]

could you please add some pointers on where to start, I see we have an AccountReader trait

[rkrasiuk]

we would need to expose the interface in HashedStorage similar to HashedPostState::from_revert_range that only loads storage changes for a single account. then we would be able to initialize storage root with in-memory hashed state
e.g.

let address = ();
let hashed_address = keccack256(address);
let hashed_storage = HashedStorage::from_revert_range(address, &tx, block_range)?;
let hahed_storage_sorted = hashed_storage.into_sorted();
let root = StorageRoot::new_hashed(
    &tx,
    HashedPostStateStorageCursor::new(
        DatabaseHashedStorageCursor::new(
            tx.cursor_dup_read::<tables::HashedStorages>()?,
            hashed_address,
        ),
        Some(&hashed_storage_sorted)
    ),
    hashed_address,
).root()?

[mvares]

@rkrasiuk, in the part of balance, could we use the getBalance? (Already it's created)

[mvares]

• Waiting by Add storage_root provider function for account #9643

[rkrasiuk]

@mvares you can just retrieve account information from HistoricalStateProvider through state_at_block_* fns

[mvares]

@mvares you can just retrieve account information from HistoricalStateProvider through state_at_block_* fns

Wouldn't it be better to use the balance method from EthApiServer?

[greged93]

hey @mvares are you still working on this? otherwise I'm down to give it a go

[mvares]

@greged93 go for it

[Rjected]

This should be fixed by #10369