From 41b4f46ae1bba898c76a7ac4cf59b991fb18cd0b Mon Sep 17 00:00:00 2001
From: Matthias Seitz <matthias.seitz@outlook.de>
Date: Thu, 7 Sep 2023 14:17:18 +0200
Subject: [PATCH] fix: skip optional fields in parity trace (#4503)

---
 crates/rpc/rpc-types/src/eth/trace/parity.rs | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/crates/rpc/rpc-types/src/eth/trace/parity.rs b/crates/rpc/rpc-types/src/eth/trace/parity.rs
index ecb73c232c..64a4ba7c6c 100644
--- a/crates/rpc/rpc-types/src/eth/trace/parity.rs
+++ b/crates/rpc/rpc-types/src/eth/trace/parity.rs
@@ -267,14 +267,18 @@ pub struct LocalizedTransactionTrace {
     /// Hash of the block, if not pending
     ///
     /// Note: this deviates from <https://openethereum.github.io/JSONRPC-trace-module#trace_transaction> which always returns a block number
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub block_hash: Option<H256>,
     /// Block number the transaction is included in, None if pending.
     ///
     /// Note: this deviates from <https://openethereum.github.io/JSONRPC-trace-module#trace_transaction> which always returns a block number
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub block_number: Option<u64>,
     /// Hash of the transaction
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub transaction_hash: Option<H256>,
     /// Transaction index within the block, None if pending.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub transaction_position: Option<u64>,
 }