fix: solve bad block from wrong transactions root hash and blob sidecar existence validation

[ulbqb]

Proposed changes

Issues

• Wrong Header.TxHash is calculated because transaction RLP contains sidecars.
• This validation causes bad blocks when CNs give blocks with sidecars to InsertChain() and PNs and ENs accidentally receive blocks with sidecars.
• Blocks with sidecars are accidentally sent to PNs and ENs.

CN behavior

Proposer behavior:

• They basically execute wait() -> PostChainEvents().

Non-proposer behavior:

• PostChainEvents
  • Instead of wait() -> PostChainEvents(), they only execute InsertChain() -> PostChainEvents().
  • PostChainEvents() need to be called to save sidecars.
• InsertChain
  • Either one of the following paths is executed:
    • They sometimes do InsertChain(blockWithSidecar in proposal) upon commit.
    • They sometimes do InsertChain(blockWithoutSidecar in NewBlockMsg) upon receiving new block from other CNs (i.e., NewBlockMsg was faster than commit).

Changes

This PR contains the following changes:

• Add DeriveTransactionsRoot for safely calculating transactions root
• Change name DeriveSha into DeriveReceiptsRoot: DeriveSha is now exclusively for receipts root.
• Remove sidecar existence validation: Non proposer CNs always execute InsertChain(). CN's blocks often have sidecars. We need to remove this.
• Controlling sending blocks over NewBlockMsg in lower level and allow sending blocks with sidecars to only CNs: Non proposer CNs sometimes do InsertChain(block from NewBlockMsg). They need sidecars over NewBlockMsg. Moreover this can control p2p message not to send sidecars to PNs and ENs.
• Update peer selection for BlobSidecarsRequestMsg: Peers are selected in the order of CN, PN, and EN. There's a good chance peers will have a sidecar in that order.

Types of changes

• 🐛 Bug fix
• ✨ Non-hardfork changes (node upgrade not required)
• 💥 Hardfork / consensus-breaking changes
• 🧪 Test improvements
• 🧰 CI / build tool
• ♻️ Chore / Refactor / Non-functional changes

Checklist

• 📖 I have read the CONTRIBUTING GUIDELINES doc
• 📝 I have signed in the PR comment I have read the CLA Document and I hereby sign the CLA in first time contribute after having read CLA
• 🟢 Lint and unit tests pass locally with my changes ($ make test)

Related issues

Further comments