From cf5eac59342a4c56b8438b3201d7071940e28b29 Mon Sep 17 00:00:00 2001 From: Yash Atreya <44857776+yash-atreya@users.noreply.github.com> Date: Wed, 3 Apr 2024 16:56:02 -0400 Subject: [PATCH] consensus: mark envelopes non-exhaustive --- crates/consensus/src/receipt/envelope.rs | 1 + crates/consensus/src/transaction/envelope.rs | 1 + crates/rpc-types/src/eth/transaction/receipt.rs | 1 + 3 files changed, 3 insertions(+) diff --git a/crates/consensus/src/receipt/envelope.rs b/crates/consensus/src/receipt/envelope.rs index aadd5bd5625..0738a0308e4 100644 --- a/crates/consensus/src/receipt/envelope.rs +++ b/crates/consensus/src/receipt/envelope.rs @@ -16,6 +16,7 @@ use alloy_rlp::{length_of_length, BufMut, Decodable, Encodable}; #[derive(Debug, Clone, PartialEq, Eq)] #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))] #[cfg_attr(feature = "serde", serde(tag = "type"))] +#[non_exhaustive] pub enum ReceiptEnvelope { /// Receipt envelope with no type flag. #[cfg_attr(feature = "serde", serde(rename = "0x0", alias = "0x00"))] diff --git a/crates/consensus/src/transaction/envelope.rs b/crates/consensus/src/transaction/envelope.rs index 82a4a2a7f47..cd9f77e175f 100644 --- a/crates/consensus/src/transaction/envelope.rs +++ b/crates/consensus/src/transaction/envelope.rs @@ -64,6 +64,7 @@ impl TryFrom for TxType { #[derive(Debug, Clone, PartialEq, Eq)] #[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))] #[cfg_attr(feature = "serde", serde(tag = "type"))] +#[non_exhaustive] pub enum TxEnvelope { /// An untagged [`TxLegacy`]. #[cfg_attr(feature = "serde", serde(rename = "0x0", alias = "0x00"))] diff --git a/crates/rpc-types/src/eth/transaction/receipt.rs b/crates/rpc-types/src/eth/transaction/receipt.rs index 1880f8deb21..1ed7effd97f 100644 --- a/crates/rpc-types/src/eth/transaction/receipt.rs +++ b/crates/rpc-types/src/eth/transaction/receipt.rs @@ -70,6 +70,7 @@ impl TransactionReceipt { | ReceiptEnvelope::Eip2930(receipt) | ReceiptEnvelope::Eip4844(receipt) | ReceiptEnvelope::Legacy(receipt) => receipt.receipt.status, + _ => false, } }