Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

poc(XCM): Deny Nested XCM Barriers #7351

Merged

Conversation

raymondkfcheung
Copy link

@raymondkfcheung raymondkfcheung commented Jan 27, 2025

Resolves (partially): #7148
Depends on: #7169, #7200

Description

For context and additional information, please refer to #7148 and #7200.

TODOs

POC Top-Level Denial Nested Denial Try Allow Remark
DenyThenTry Blocks top-level instructions only.
RecursiveDenyThenTry Blocks both top-level and nested instructions.
DenyInstructionsWithXcm Focuses on nested instructions, requires additional checks for top-level denial.
DenyFirstInstructionsWithXcm Prioritises top-level denial before recursive checks.

yrong and others added 30 commits January 15, 2025 10:42
Co-authored-by: Francisco Aguirre <franciscoaguirreperez@gmail.com>
Co-authored-by: Clara van Staden <claravanstaden64@gmail.com>
Co-authored-by: Clara van Staden <claravanstaden64@gmail.com>
@raymondkfcheung raymondkfcheung marked this pull request as ready for review January 30, 2025 20:20
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13070363687
Failed job name: fmt

@raymondkfcheung raymondkfcheung merged commit df64f5f into bko-deny-nested-xcm-barrier Jan 31, 2025
200 of 206 checks passed
@raymondkfcheung raymondkfcheung deleted the ray-deny-nested-xcm-barrier branch January 31, 2025 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C1-mentor A task where a mentor is available. Please indicate in the issue who the mentor could be. C2-good-first-issue A task for a first time contributor to become familiar with the Polkadot-SDK. T6-XCM This PR/Issue is related to XCM.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants