refactor: Rename SignedAuthorizationListEntry to AuthorizationListEntry

Author: pgherveou

substrate/frame/revive/src/evm/api/rlp_codec.rs

@@ -234,7 +234,7 @@ impl Decodable for UnsignedAuthorizationListEntry {
 	}
 }
 
-impl Encodable for SignedAuthorizationListEntry {
+impl Encodable for AuthorizationListEntry {
 	fn rlp_append(&self, s: &mut rlp::RlpStream) {
 		s.begin_list(6);
 		s.append(&self.authorization_unsigned.chain_id);
@@ -246,9 +246,9 @@ impl Encodable for SignedAuthorizationListEntry {
 	}
 }
 
-impl Decodable for SignedAuthorizationListEntry {
+impl Decodable for AuthorizationListEntry {
 	fn decode(rlp: &rlp::Rlp) -> Result<Self, rlp::DecoderError> {
-		Ok(SignedAuthorizationListEntry {
+		Ok(AuthorizationListEntry {
 			authorization_unsigned: UnsignedAuthorizationListEntry {
 				chain_id: rlp.val_at(0)?,
 				address: rlp.val_at(1)?,

substrate/frame/revive/src/evm/api/rpc_types.rs

@@ -459,7 +459,7 @@ fn from_unsigned_works_for_7702() {
 		gas_price: U256::from(20),
 		max_fee_per_gas: U256::from(20),
 		max_priority_fee_per_gas: U256::from(1),
-		authorization_list: vec![SignedAuthorizationListEntry {
+		authorization_list: vec![AuthorizationListEntry {
 			authorization_unsigned: UnsignedAuthorizationListEntry {
 				chain_id: U256::from(1),
 				address: H160::from_low_u64_be(42),

substrate/frame/revive/src/evm/api/rpc_types_gen.rs

@@ -257,7 +257,7 @@ pub struct GenericTransaction {
 	/// authorizationList
 	/// List of account code authorizations (EIP-7702)
 	#[serde(default, skip_serializing_if = "Vec::is_empty")]
-	pub authorization_list: Vec<SignedAuthorizationListEntry>,
+	pub authorization_list: Vec<AuthorizationListEntry>,
 	/// blobVersionedHashes
 	/// List of versioned blob hashes associated with the transaction's EIP-4844 data blobs.
 	#[serde(default)]
@@ -787,7 +787,7 @@ pub struct Transaction7702Unsigned {
 	pub access_list: AccessList,
 	/// authorizationList
 	/// List of account code authorizations
-	pub authorization_list: Vec<SignedAuthorizationListEntry>,
+	pub authorization_list: Vec<AuthorizationListEntry>,
 	/// chainId
 	/// Chain ID that this transaction is valid on.
 	pub chain_id: U256,
@@ -862,7 +862,7 @@ pub struct UnsignedAuthorizationListEntry {
 	DecodeWithMemTracking,
 )]
 #[serde(rename_all = "camelCase")]
-pub struct SignedAuthorizationListEntry {
+pub struct AuthorizationListEntry {
 	/// The unsigned authorization entry
 	#[serde(flatten)]
 	pub authorization_unsigned: UnsignedAuthorizationListEntry,
@@ -874,7 +874,7 @@ pub struct SignedAuthorizationListEntry {
 	pub s: U256,
 }
 
-impl SignedAuthorizationListEntry {
+impl AuthorizationListEntry {
 	/// Convert signature components (r, s, y_parity) into a 65-byte ECDSA signature.
 	///
 	/// # Returns

substrate/frame/revive/src/evm/call.rs

@@ -49,7 +49,7 @@ pub struct CallInfo<T: Config> {
 	/// The ethereum gas limit of the transaction.
 	pub eth_gas_limit: U256,
 	/// EIP-7702: List of authorization tuples to process
-	pub authorization_list: Vec<crate::evm::SignedAuthorizationListEntry>,
+	pub authorization_list: Vec<crate::evm::AuthorizationListEntry>,
 }
 
 /// Mode for creating a call from an ethereum transaction.

substrate/frame/revive/src/evm/eip7702.rs

@@ -23,7 +23,7 @@
 
 use crate::{
 	address::AddressMapper,
-	evm::api::{recover_eth_address_from_message, rlp, SignedAuthorizationListEntry},
+	evm::api::{recover_eth_address_from_message, rlp, AuthorizationListEntry},
 	storage::AccountInfo,
 	weights::WeightInfo,
 	Config,
@@ -58,7 +58,7 @@ pub const EIP7702_MAGIC: u8 = 0x05;
 /// # Returns
 /// `Ok(())` on success, or `Err` if out of weight
 pub fn process_authorizations<T: Config>(
-	authorization_list: &[SignedAuthorizationListEntry],
+	authorization_list: &[AuthorizationListEntry],
 	chain_id: U256,
 	meter: &mut WeightMeter,
 ) -> DispatchResult {
@@ -94,7 +94,7 @@ pub fn process_authorizations<T: Config>(
 /// Returns the authority address if validation succeeds, None otherwise.
 /// This is exposed for benchmarking purposes.
 pub(crate) fn validate_authorization<T: Config>(
-	auth: &SignedAuthorizationListEntry,
+	auth: &AuthorizationListEntry,
 	chain_id: U256,
 ) -> Option<H160> {
 	// Validate chain_id
@@ -180,7 +180,7 @@ pub(crate) fn apply_delegation<T: Config>(authority: &H160, target_address: H160
 ///
 /// Implements the EIP-7702 signature recovery:
 /// - Message = keccak(MAGIC || rlp([chain_id, address, nonce]))
-fn recover_authority(auth: &SignedAuthorizationListEntry) -> Result<H160, ()> {
+fn recover_authority(auth: &AuthorizationListEntry) -> Result<H160, ()> {
 	let mut message = Vec::new();
 	message.push(EIP7702_MAGIC);
 
@@ -198,7 +198,7 @@ fn recover_authority(auth: &SignedAuthorizationListEntry) -> Result<H160, ()> {
 pub(crate) fn sign_authorization(
 	signing_key: &k256::ecdsa::SigningKey,
 	authorization_unsigned: crate::evm::UnsignedAuthorizationListEntry,
-) -> SignedAuthorizationListEntry {
+) -> AuthorizationListEntry {
 	use sp_core::keccak_256;
 
 	let mut message = Vec::new();
@@ -209,7 +209,7 @@ pub(crate) fn sign_authorization(
 	let (signature, recovery_id) =
 		signing_key.sign_prehash_recoverable(&hash).expect("signing succeeds");
 
-	SignedAuthorizationListEntry {
+	AuthorizationListEntry {
 		authorization_unsigned,
 		y_parity: U256::from(recovery_id.to_byte()),
 		r: U256::from_big_endian(&signature.r().to_bytes()),

substrate/frame/revive/src/lib.rs

@@ -1384,7 +1384,7 @@ pub mod pallet {
 			transaction_encoded: Vec<u8>,
 			effective_gas_price: U256,
 			encoded_len: u32,
-			authorization_list: Vec<evm::SignedAuthorizationListEntry>,
+			authorization_list: Vec<evm::AuthorizationListEntry>,
 		) -> DispatchResultWithPostInfo {
 			let signer = Self::ensure_eth_signed(origin)?;
 			let origin = OriginFor::<T>::signed(signer.clone());

substrate/frame/revive/src/test_utils/builder.rs

@@ -286,7 +286,7 @@ builder!(
 		transaction_encoded: Vec<u8>,
 		effective_gas_price: U256,
 		encoded_len: u32,
-		authorization_list: Vec<crate::evm::SignedAuthorizationListEntry>,
+		authorization_list: Vec<crate::evm::AuthorizationListEntry>,
 	) -> DispatchResultWithPostInfo;
 
 	/// Create a [`EthCallWithAuthorizationListBuilder`] with default values.

substrate/frame/revive/src/tests/eip7702.rs

@@ -18,7 +18,7 @@
 //! Tests for EIP-7702: Set EOA Account Code
 
 use crate::{
-	evm::{fees::InfoT, SignedAuthorizationListEntry, UnsignedAuthorizationListEntry},
+	evm::{fees::InfoT, AuthorizationListEntry, UnsignedAuthorizationListEntry},
 	storage::AccountInfo,
 	test_utils::builder::Contract,
 	tests::{builder, *},
@@ -74,7 +74,7 @@ impl TestSigner {
 		chain_id: U256,
 		address: H160,
 		nonce: U256,
-	) -> SignedAuthorizationListEntry {
+	) -> AuthorizationListEntry {
 		let mut message = Vec::new();
 		message.push(crate::evm::eip7702::EIP7702_MAGIC);
 
@@ -98,7 +98,7 @@ impl TestSigner {
 		let s = U256::from_big_endian(&s_bytes);
 		let y_parity = U256::from(recovery_id);
 
-		SignedAuthorizationListEntry { authorization_unsigned, y_parity, r, s }
+		AuthorizationListEntry { authorization_unsigned, y_parity, r, s }
 	}
 }