git.delta.rocks / unique-network / refs/commits / 3ab9e10ac5b6

--- a/.gitignore
+++ b/.gitignore
@@ -11,6 +11,7 @@
 *store_key*.json
 
 /.idea/
+/.cargo/
 
 tests/.vscode
 cumulus-parachain/

--- a/Cargo.lock
+++ b/Cargo.lock
@@ -6146,6 +6146,7 @@
  "sp-core",
  "sp-runtime",
  "sp-std",
+ "up-evm-mapping",
  "up-sponsorship",
 ]
 

--- a/pallets/common/src/account.rs
+++ b/pallets/common/src/account.rs
@@ -23,6 +23,7 @@
 use pallet_evm::AddressMapping;
 use sp_std::vec::Vec;
 use sp_std::clone::Clone;
+
 pub use up_evm_mapping::EvmBackwardsAddressMapping;
 
 pub trait CrossAccountId<AccountId>:

--- a/pallets/evm-contract-helpers/Cargo.toml
+++ b/pallets/evm-contract-helpers/Cargo.toml
@@ -17,6 +17,7 @@
 pallet-evm-coder-substrate = { default-features = false, path = '../../pallets/evm-coder-substrate' }
 pallet-evm = { default-features = false, git = "https://github.com/uniquenetwork/frontier.git", branch = "unique-polkadot-v0.9.18" }
 up-sponsorship = { version = "0.1.0", default-features = false, git = "https://github.com/UniqueNetwork/pallet-sponsoring", branch = 'polkadot-v0.9.18' }
+up-evm-mapping = { default-features = false, path = "../../primitives/evm-mapping" }
 log = "0.4.14"
 
 [dependencies.codec]

--- a/pallets/evm-contract-helpers/src/eth.rs
+++ b/pallets/evm-contract-helpers/src/eth.rs
@@ -17,14 +17,17 @@
 use core::marker::PhantomData;
 use evm_coder::{abi::AbiWriter, execution::Result, generate_stubgen, solidity_interface, types::*};
 use pallet_evm_coder_substrate::{SubstrateRecorder, WithRecorder};
-use pallet_evm::{ExitRevert, OnCreate, OnMethodCall, PrecompileResult, PrecompileFailure};
+use pallet_evm::{
+	ExitRevert, OnCreate, OnMethodCall, PrecompileResult, PrecompileFailure, AddressMapping
+};
 use sp_core::H160;
 use crate::{
 	AllowlistEnabled, Config, Owner, Pallet, SponsorBasket, SponsoringRateLimit, SponsoringModeT,
 };
 use frame_support::traits::Get;
 use up_sponsorship::SponsorshipHandler;
-use sp_std::{convert::TryInto, vec::Vec};
+use up_evm_mapping::EvmBackwardsAddressMapping;
+use sp_std::vec::Vec;
 
 struct ContractHelpers<T: Config>(SubstrateRecorder<T>);
 impl<T: Config> WithRecorder<T> for ContractHelpers<T> {
@@ -177,13 +180,15 @@
 }
 
 pub struct HelpersContractSponsoring<T: Config>(PhantomData<*const T>);
-impl<T: Config> SponsorshipHandler<H160, (H160, Vec<u8>)> for HelpersContractSponsoring<T> {
-	fn get_sponsor(who: &H160, call: &(H160, Vec<u8>)) -> Option<H160> {
+impl<T: Config> SponsorshipHandler<T::AccountId, (H160, Vec<u8>)> for HelpersContractSponsoring<T> {
+	fn get_sponsor(who: &T::AccountId, call: &(H160, Vec<u8>)) -> Option<T::AccountId> {
 		let mode = <Pallet<T>>::sponsoring_mode(call.0);
 		if mode == SponsoringModeT::Disabled {
 			return None;
 		}
-		if mode == SponsoringModeT::Allowlisted && !<Pallet<T>>::allowed(call.0, *who) {
+
+		let who = T::EvmBackwardsAddressMapping::from_account_id(who.clone());
+		if mode == SponsoringModeT::Allowlisted && !<Pallet<T>>::allowed(call.0, who) {
 			return None;
 		}
 		let block_number = <frame_system::Pallet<T>>::block_number() as T::BlockNumber;
@@ -199,7 +204,8 @@
 
 		<SponsorBasket<T>>::insert(&call.0, who, block_number);
 
-		Some(call.0)
+		let sponsor = T::EvmAddressMapping::into_account_id(call.0);
+		Some(sponsor)
 	}
 }
 

--- a/pallets/evm-contract-helpers/src/lib.rs
+++ b/pallets/evm-contract-helpers/src/lib.rs
@@ -33,6 +33,8 @@
 	pub trait Config: frame_system::Config + pallet_evm_coder_substrate::Config {
 		type ContractAddress: Get<H160>;
 		type DefaultSponsoringRateLimit: Get<Self::BlockNumber>;
+		type EvmAddressMapping: pallet_evm::AddressMapping<Self::AccountId>;
+		type EvmBackwardsAddressMapping: up_evm_mapping::EvmBackwardsAddressMapping<Self::AccountId>;
 	}
 
 	#[pallet::error]

3636
37	#[pallet::config]37	#[pallet::config]
38	pub trait Config: frame_system::Config {38	pub trait Config: frame_system::Config {
39		type EvmSponsorshipHandler: SponsorshipHandler<H160, (H160, Vec<u8>)>;39		type EvmSponsorshipHandler: SponsorshipHandler<Self::AccountId, (H160, Vec<u8>)>;
40		type Currency: Currency<Self::AccountId>;40		type Currency: Currency<Self::AccountId>;
41		type EvmBackwardsAddressMapping: EvmBackwardsAddressMapping<Self::AccountId>;41		type EvmBackwardsAddressMapping: EvmBackwardsAddressMapping<Self::AccountId>;
42		type EvmAddressMapping: AddressMapping<Self::AccountId>;42		type EvmAddressMapping: AddressMapping<Self::AccountId>;
60}60}
6161
62pub struct TransactionValidityHack<T: Config>(PhantomData<*const T>);62pub struct TransactionValidityHack<T: Config>(PhantomData<*const T>);
63impl<T: Config> fp_evm::TransactionValidityHack for TransactionValidityHack<T> {63impl<T: Config> fp_evm::TransactionValidityHack<T::AccountId> for TransactionValidityHack<T> {
64	fn who_pays_fee(origin: H160, reason: &WithdrawReason) -> Option<H160> {64	fn who_pays_fee(origin: H160, reason: &WithdrawReason) -> Option<T::AccountId> {
65		match reason {65		match reason {
66			WithdrawReason::Call { target, input } => {66			WithdrawReason::Call { target, input } => {
67				// This method is only used for checking, we shouldn't touch storage in it67				// This method is only used for checking, we shouldn't touch storage in it
68				frame_support::storage::with_transaction(|| {68				frame_support::storage::with_transaction(|| {
69					let origin_sub = T::EvmAddressMapping::into_account_id(origin);
69					TransactionOutcome::Rollback(T::EvmSponsorshipHandler::get_sponsor(70					TransactionOutcome::Rollback(T::EvmSponsorshipHandler::get_sponsor(
70						&origin,71						&origin_sub,
71						&(*target, input.clone()),72						&(*target, input.clone()),
72					))73					))
73				})74				})
85	type LiquidityInfo = Option<ChargeEvmLiquidityInfo<T>>;86	type LiquidityInfo = Option<ChargeEvmLiquidityInfo<T>>;
8687
87	fn withdraw_fee(88	fn withdraw_fee(
88		who: &H160,89		who: &T::AccountId,
89		reason: WithdrawReason,90		reason: WithdrawReason,
90		fee: U256,91		fee: U256,
91	) -> core::result::Result<Self::LiquidityInfo, pallet_evm::Error<T>> {92	) -> core::result::Result<Self::LiquidityInfo, pallet_evm::Error<T>> {
92		let mut who_pays_fee = *who;93		let mut who_pays_fee = who.clone();
93		if let WithdrawReason::Call { target, input } = &reason {94		if let WithdrawReason::Call { target, input } = &reason {
94			who_pays_fee = T::EvmSponsorshipHandler::get_sponsor(who, &(*target, input.clone()))95			who_pays_fee = T::EvmSponsorshipHandler::get_sponsor(who, &(*target, input.clone()))
95				.unwrap_or(who_pays_fee);96				.unwrap_or(who_pays_fee);
100			fee,102			fee,
101		)?;103		)?;
104
105		let who_pays_fee_eth = T::EvmBackwardsAddressMapping::from_account_id(who_pays_fee);
102		Ok(negative_imbalance.map(|i| ChargeEvmLiquidityInfo {106		Ok(negative_imbalance.map(|i| ChargeEvmLiquidityInfo {
103			who: who_pays_fee,107			who: who_pays_fee_eth,
104			negative_imbalance: i,108			negative_imbalance: i,
105		}))109		}))
106	}110	}
107111
108	fn correct_and_deposit_fee(112	fn correct_and_deposit_fee(
109		who: &H160,113		who: &T::AccountId,
110		corrected_fee: U256,114		corrected_fee: U256,
111		already_withdrawn: Self::LiquidityInfo,115		already_withdrawn: Self::LiquidityInfo,
112	) {116	) {
113		<EVMCurrencyAdapter<<T as Config>::Currency, ()> as pallet_evm::OnChargeEVMTransaction<T>>::correct_and_deposit_fee(117		<EVMCurrencyAdapter<<T as Config>::Currency, ()> as pallet_evm::OnChargeEVMTransaction<T>>::correct_and_deposit_fee(
114			&already_withdrawn.as_ref().map(|e| e.who).unwrap_or(*who),118			&already_withdrawn.as_ref().map(|e| T::EvmAddressMapping::into_account_id(e.who)).unwrap_or(who.clone()),
115			corrected_fee,119			corrected_fee,
116			already_withdrawn.map(|e| e.negative_imbalance),120			already_withdrawn.map(|e| e.negative_imbalance),
117		)121		)
142					<frame_system::RawOrigin<T::AccountId>>::Signed(who.clone()).into(),146					<frame_system::RawOrigin<T::AccountId>>::Signed(who.clone()).into(),
143				)147				)
144				.ok()?;148				.ok()?;
145				let who = T::EvmBackwardsAddressMapping::from_account_id(who.clone());
146				// Effects from EvmSponsorshipHandler are applied in OnChargeEvmTransaction by pallet_evm::runner149				// Effects from EvmSponsorshipHandler are applied in OnChargeEvmTransaction by pallet_evm::runner
147				// TODO: Should we implement simulation mode (test, but do not apply effects) in `up-sponsorship`?150				// TODO: Should we implement simulation mode (test, but do not apply effects) in `up-sponsorship`?
148				let sponsor = frame_support::storage::with_transaction(|| {151				let sponsor = frame_support::storage::with_transaction(|| {
151						&(*target, input.clone()),154						&(*target, input.clone()),
152					))155					))
153				})?;156				})?;
154				let sponsor = T::EvmAddressMapping::into_account_id(sponsor);
155				Some(sponsor)157				Some(sponsor)
156			}158			}
157			_ => None,159			_ => None,

--- a/pallets/unique/src/eth/sponsoring.rs
+++ b/pallets/unique/src/eth/sponsoring.rs
@@ -32,14 +32,12 @@
 use pallet_fungible::erc::{UniqueFungibleCall, ERC20Call};
 
 pub struct UniqueEthSponsorshipHandler<T: Config>(PhantomData<*const T>);
-impl<T: Config> SponsorshipHandler<H160, (H160, Vec<u8>)> for UniqueEthSponsorshipHandler<T> {
-	fn get_sponsor(who: &H160, call: &(H160, Vec<u8>)) -> Option<H160> {
+impl<T: Config> SponsorshipHandler<T::AccountId, (H160, Vec<u8>)> for UniqueEthSponsorshipHandler<T> {
+	fn get_sponsor(who: &T::AccountId, call: &(H160, Vec<u8>)) -> Option<T::AccountId> {
 		let collection_id = map_eth_to_id(&call.0)?;
 		let collection = <CollectionHandle<T>>::new(collection_id)?;
 		let sponsor = collection.sponsorship.sponsor()?.clone();
-		let sponsor =
-			<T as pallet_common::Config>::EvmBackwardsAddressMapping::from_account_id(sponsor);
-		let who = T::CrossAccountId::from_eth(*who);
+		let who = T::CrossAccountId::from_sub(who.clone());
 		let (method_id, mut reader) = AbiReader::new_call(&call.1).ok()?;
 		match &collection.mode {
 			crate::CollectionMode::NFT => {

--- a/runtime/opal/src/lib.rs
+++ b/runtime/opal/src/lib.rs
@@ -933,6 +933,8 @@
 impl pallet_evm_contract_helpers::Config for Runtime {
 	type ContractAddress = HelpersContractAddress;
 	type DefaultSponsoringRateLimit = DefaultSponsoringRateLimit;
+	type EvmAddressMapping = pallet_evm::HashedAddressMapping<Self::Hashing>;
+	type EvmBackwardsAddressMapping = up_evm_mapping::MapBackwardsAddressTruncated;
 }
 
 construct_runtime!(