Support multiple all eth protocol versions announced in HelloMessage

[mattsse]

Describe the feature

as highlighted here:
https://t.me/paradigm_reth/1611

we're announcing support for eth66/67 by default in Hello but only check for a single version in the eth Status handshake.

reth/crates/net/eth-wire/src/ethstream.rs

Lines 101 to 106 in e0dbcae

	if status.version != resp.version {
	return Err(EthHandshakeError::MismatchedProtocolVersion {
	expected: status.version,
	got: resp.version,
	}
	.into())

the status message should also accept older versions if announced.

TODO

if we add this to Status directly, we can no longer derive Rlp
so perhaps we add a list of versions as a separate parameter to the handshake function

wdyt @Rjected

Additional context

No response

[Rjected]

I think this is because we always use EthVersion::Eth67 in the Default impl of Status:

reth/crates/net/eth-wire/src/types/status.rs

Lines 152 to 158 in e0dbcae

	impl Default for Status {
	fn default() -> Self {
	Status {
	version: EthVersion::Eth67 as u8,
	chain: Chain::Named(ethers_core::types::Chain::Mainnet),
	total_difficulty: U256::from(17_179_869_184u64),
	blockhash: MAINNET.genesis_hash(),

Instead, this should be selected depending on the highest version shared eth capability. Ideally this would be fully solved by #791 in a way that lets us preserve the current behavior, but for now, adding a parameter in handshake seems like the best solution

[mattsse]

Instead, this should be selected depending on the highest version shared eth capability.

for the handshake this should check that the version of the remote peer's Status message is supported.

[Rjected]

for the handshake this should check that the version of the remote peer's Status message is supported.

right, yeah

[jinsankim]

I'd like to take this one.

[jinsankim]

Question: Do we need to send EthMessage per a list of versions? I think we should do but I don't have confidence.

reth/crates/net/eth-wire/src/ethstream.rs

Lines 59 to 64 in a588f79

	// we need to encode and decode here on our own because we don't have an `EthStream` yet
	// The max length for a status with TTD is: <msg id = 1 byte> + <rlp(status) = 88 byte>
	let mut our_status_bytes = BytesMut::with_capacity(1 + 88);
	ProtocolMessage::from(EthMessage::Status(status)).encode(&mut our_status_bytes);
	let our_status_bytes = our_status_bytes.freeze();
	self.inner.send(our_status_bytes).await?;