Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Create thread_local in XCM executor to limit recursion depth #6304

Merged
merged 14 commits into from
Dec 1, 2022
Merged

Create thread_local in XCM executor to limit recursion depth #6304

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
4bb4b5d
Create thread_local in XCM executor to limit recursion depth
KiChjang Nov 17, 2022
9d5b815
Add unit test for recursion limit
KiChjang Nov 21, 2022
395a099
Fix statefulness in tests
KiChjang Nov 26, 2022
85fe5c2
Remove panic
KiChjang Nov 26, 2022
661c0e1
Use defer and environmental macro
KiChjang Nov 27, 2022
d098dfd
Fix the implementation
bkchr Nov 29, 2022
49c5f34
Merge branch 'gav-xcm-v3' into kckyeung/xcm-v3-recursion-limit
KiChjang Nov 30, 2022
52de9c2
Use nicer interface
bkchr Nov 30, 2022
ab4b7aa
Merge branch 'kckyeung/xcm-v3-recursion-limit' of github.com:parityte…
bkchr Nov 30, 2022
ee0b2ac
Change ThisNetwork to AnyNetwork
KiChjang Dec 1, 2022
989a126
Move recursion check up to top level
KiChjang Dec 1, 2022
48973ba
cargo fmt
KiChjang Dec 1, 2022
8e1718b
Update comment
KiChjang Dec 1, 2022
fcb6401
Merge branch 'gav-xcm-v3' into kckyeung/xcm-v3-recursion-limit
KiChjang Dec 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Change ThisNetwork to AnyNetwork
  • Loading branch information
@KiChjang
KiChjang committed Dec 1, 2022
commit ee0b2ac5213ccdbc41e63a07d5bba7502a8800b4
6 changes: 3 additions & 3 deletions runtime/test-runtime/src/xcm_config.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ use xcm_executor::{

parameter_types! {
pub const BaseXcmWeight: xcm::latest::Weight = Weight::from_parts(1_000, 1_000);
pub const ThisNetwork: NetworkId = NetworkId::Polkadot;
pub const AnyNetwork: Option<NetworkId> = None;
pub const MaxInstructions: u32 = 100;
pub const MaxAssetsIntoHolding: u32 = 16;
pub const UniversalLocation: xcm::latest::InteriorMultiLocation = xcm::latest::Junctions::Here;
Expand All @@ -41,7 +41,7 @@ parameter_types! {
/// of this chain.
pub type LocalOriginToLocation = (
// And a usual Signed origin to be used in XCM as a corresponding AccountId32
SignedToAccountId32<crate::RuntimeOrigin, crate::AccountId, ThisNetwork>,
SignedToAccountId32<crate::RuntimeOrigin, crate::AccountId, AnyNetwork>,
);

pub struct DoNothingRouter;
Expand Down Expand Up @@ -86,7 +86,7 @@ impl WeightTrader for DummyWeightTrader {

type OriginConverter = (
pallet_xcm::XcmPassthrough<super::RuntimeOrigin>,
SignedAccountId32AsNative<ThisNetwork, super::RuntimeOrigin>,
SignedAccountId32AsNative<AnyNetwork, super::RuntimeOrigin>,
);

pub struct XcmConfig;
Expand Down