remove transaction forwarder trait

[mattsse]

Describe the feature

now that we have separate ethapi impls for op and eth, we no longer need this

reth/crates/rpc/rpc-eth-api/src/helpers/transaction.rs

Lines 651 to 657 in fb6ea8b

	pub trait RawTransactionForwarder: fmt::Debug + Send + Sync + 'static {
	/// Forwards raw transaction bytes for `eth_sendRawTransaction`
	async fn forward_raw_transaction(&self, raw: &[u8]) -> EthResult<()>;
	}
	/// Configure server's forwarder for `eth_sendRawTransaction`, at runtime.
	pub trait UpdateRawTxForwarder {

TODO

• remove traits and field in EthAPI

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

  Line 56 in fb6ea8b

  raw_transaction_forwarder: Option<Arc<dyn RawTransactionForwarder>>,

• integrate Sequencerclient in OpEthApi instead

  reth/crates/optimism/node/src/rpc.rs

  Lines 34 to 38 in fb6ea8b

  	/// A client to interact with a Sequencer
  	#[derive(Debug, Clone)]
  	pub struct SequencerClient {
  	inner: Arc<SequencerClientInner>,
  	}

  
  

  reth/crates/optimism/rpc/src/eth/mod.rs

  Line 43 in fb6ea8b

  pub struct OpEthApi<Eth> {

Additional context

No response

[nkysg]

May I take this?

[EmmanuelAR]

Hello @mattsse

My name is Emmanuel, a software developer from Costa Rica. I would be happy to assist with this task. I will review the current implementation and will guide by the comments that you post in the ticket

Please let me know if this is acceptable.

Best regards,
Emmanuel 😁