Skip to content

[pull] master from paritytech:master#654

Merged
pull[bot] merged 4 commits intodhiway:masterfrom
paritytech:master
May 8, 2026
Merged

[pull] master from paritytech:master#654
pull[bot] merged 4 commits intodhiway:masterfrom
paritytech:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented May 8, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

alvicsam and others added 4 commits May 8, 2026 12:01
The inbound-queue v2 message converter falls back to the Snowbridge
sovereign account on AssetHub as the asset claimer when no explicit
claimer is supplied. Previously this fallback used `AccountId32 {
network: None, .. }`, which did not match the location AssetHub's
signed-origin converter produces (it sets `network:
Some(LocalNetwork)`). The trap-key hash stored on `AssetsTrapped`
therefore could not be matched by a signed `polkadotXcm.claim_assets`
call, making default-claimer trapped funds effectively unrecoverable
without a runtime upgrade.

This PR sets `network: Some(LocalNetwork::get())` on the fallback
claimer so its `Location` agrees with what `SignedToAccountId32<_, _,
LocalNetwork>` yields on AssetHub, and adds a test covering the
no-claimer-supplied path.

---------

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
# Description

Just as the title says.

---------

Signed-off-by: georgepisaltu <george.pisaltu@parity.io>
Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@pull pull Bot locked and limited conversation to collaborators May 8, 2026
@pull pull Bot added the ⤵️ pull label May 8, 2026
@pull pull Bot merged commit bcd139f into dhiway:master May 8, 2026
@pull pull Bot deployed to master May 8, 2026 16:39 Active
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants