fix: estimate: auth list should be send unchecked to the runtime

The runtime api expect type `Vec<AuthorizationListItem>`, the client should pass the authorization list without any pre-conversion

Author: librelois

client/rpc-core/src/types/transaction_request.rs

@@ -269,7 +269,7 @@ impl From<TransactionRequest> for Option<TransactionMessage> {
 				value,
 				data: data_bytes,
 				access_list,
-				authorization_list: req.authorization_list.unwrap(),
+				authorization_list: req.authorization_list.unwrap_or_default(),
 				chain_id,
 			})),
 			// EIP1559: Has max_fee_per_gas but no gas_price, or all fee fields are None

client/rpc/src/eth/execute.rs

@@ -443,19 +443,6 @@ where
 						.map(|item| (item.address, item.storage_keys))
 						.collect::<Vec<(sp_core::H160, Vec<H256>)>>();
 
-					let authorization_list = authorization_list
-						.unwrap_or_default()
-						.iter()
-						.map(|d| {
-							(
-								U256::from(d.chain_id),
-								d.address,
-								d.nonce,
-								d.authorizing_address().ok(),
-							)
-						})
-						.collect::<Vec<(U256, sp_core::H160, U256, Option<sp_core::H160>)>>();
-
 					let encoded_params = Encode::encode(&(
 						&from.unwrap_or_default(),
 						&to,
@@ -467,7 +454,7 @@ where
 						&nonce,
 						&false,
 						&Some(access_list),
-						&Some(authorization_list),
+						&authorization_list,
 					));
 					let overlayed_changes = self.create_overrides_overlay(
 						substrate_hash,
@@ -948,19 +935,6 @@ where
 								.map(|item| (item.address, item.storage_keys))
 								.collect::<Vec<(sp_core::H160, Vec<H256>)>>();
 
-							let authorization_list = authorization_list
-								.unwrap_or_default()
-								.iter()
-								.map(|d| {
-									(
-										U256::from(d.chain_id),
-										d.address,
-										d.nonce,
-										d.authorizing_address().ok(),
-									)
-								})
-								.collect::<Vec<(U256, H160, U256, Option<H160>)>>();
-
 							let encoded_params = Encode::encode(&(
 								&from.unwrap_or_default(),
 								&to,
@@ -974,7 +948,7 @@ where
 								&Some(
 									access_list
 								),
-								&Some(authorization_list),
+								&authorization_list,
 							));
 
 							// Proof size recording
@@ -1136,19 +1110,6 @@ where
 								.map(|item| (item.address, item.storage_keys))
 								.collect::<Vec<(sp_core::H160, Vec<H256>)>>();
 
-							let authorization_list = authorization_list
-								.unwrap_or_default()
-								.iter()
-								.map(|d| {
-									(
-										U256::from(d.chain_id),
-										d.address,
-										d.nonce,
-										d.authorizing_address().ok(),
-									)
-								})
-								.collect::<Vec<(U256, H160, U256, Option<H160>)>>();
-
 							let encoded_params = Encode::encode(&(
 								&from.unwrap_or_default(),
 								&data,
@@ -1161,7 +1122,7 @@ where
 								&Some(
 									access_list
 								),
-								&Some(authorization_list),
+								&authorization_list,
 							));
 
 							// Enable proof size recording