fix: ErrExecutionReverted JSONRPCError conversion error with Lotus Gateway (#6551)

Author: sudo-shashank

CHANGELOG.md

@@ -37,6 +37,8 @@
 
 ### Fixed
 
+- [#6551](https://github.com/ChainSafe/forest/pull/6551): Fixed `ErrExecutionReverted` JSONRPCError conversion error with Lotus Gateway.
+
 ## Forest v0.32.0 "Ember"
 
 This is a non-mandatory release for all node operators. It resets F3 on calibnet, also includes new V2 RPC methods, a few bug fixes and `lotus-gateway` compatibility fixes.

src/rpc/methods/eth/errors.rs

@@ -14,10 +14,7 @@ pub const EXECUTION_REVERTED_CODE: i32 = 3;
 #[derive(Clone, Debug, Error, Serialize)]
 pub enum EthErrors {
     #[error("{message}")]
-    ExecutionReverted {
-        message: String,
-        data: Option<String>,
-    },
+    ExecutionReverted { message: String, data: String },
 }
 
 impl EthErrors {
@@ -33,7 +30,7 @@ impl EthErrors {
             message: format!(
                 "message execution failed (exit=[{exit_code}]{revert_reason}, vm error=[{error}])"
             ),
-            data: (!data.is_empty()).then(|| format!("0x{}", hex::encode(data))),
+            data: format!("0x{}", hex::encode(data)),
         }
     }
 }
@@ -54,7 +51,7 @@ impl RpcErrorData for EthErrors {
     fn error_data(&self) -> Option<serde_json::Value> {
         match self {
             EthErrors::ExecutionReverted { data, .. } => {
-                data.clone().map(serde_json::Value::String)
+                Some(serde_json::Value::String(data.clone()))
             }
         }
     }