lighthouse icon indicating copy to clipboard operation
lighthouse copied to clipboard

Drop verify single RpcBlock fns

Open dapplion opened this issue 8 months ago • 4 comments

Issue Addressed

Noted that we never have to verify individual RpcBlocks for sync. Lookup sync sends the block alone always, and range sync sends chain segments.

Proposed Changes

  • Removed verify_kzg_for_rpc_block
  • Removed IntoExecutionPendingBlock impl for RpcBlock. Now lookup sync blocks use the SignedBeaconBlock IntoExecutionPendingBlock impl which is the same
  • Switched tests to import RpcBlocks with process_chain_segment

dapplion avatar Feb 18 '25 04:02 dapplion