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

[XCM] Treat recursion limit error as transient in the MQ #4202

Merged
merged 9 commits into from
Apr 25, 2024

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Apr 18, 2024

Changes:

  • Add new error variant ProcessMessageError::StackLimitReached and treat XCM error ExceedsStackLimit as such.

@ggwpez ggwpez requested review from a team as code owners April 18, 2024 15:51
@ggwpez ggwpez added T6-XCM This PR/Issue is related to XCM. T2-pallets This PR/Issue is related to a particular pallet. labels Apr 18, 2024
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@paritytech-review-bot paritytech-review-bot bot requested a review from a team April 18, 2024 20:17
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
polkadot/xcm/xcm-executor/src/lib.rs Outdated Show resolved Hide resolved
ggwpez and others added 3 commits April 19, 2024 12:30
Co-authored-by: Branislav Kontur <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@ggwpez ggwpez added this pull request to the merge queue Apr 25, 2024
Merged via the queue into master with commit 0770417 Apr 25, 2024
134 of 138 checks passed
@ggwpez ggwpez deleted the oty-transient-mq-xcm-error branch April 25, 2024 09:29
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
…4202)

Changes:
- Add new error variant `ProcessMessageError::StackLimitReached` and
treat XCM error `ExceedsStackLimit` as such.

---------

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Co-authored-by: Branislav Kontur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T2-pallets This PR/Issue is related to a particular pallet. T6-XCM This PR/Issue is related to XCM.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants