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

Backward incompatibility between XCM v0 and v1 #4056

Closed
xlc opened this issue Oct 11, 2021 · 2 comments
Closed

Backward incompatibility between XCM v0 and v1 #4056

xlc opened this issue Oct 11, 2021 · 2 comments

Comments

@xlc
Copy link
Contributor

xlc commented Oct 11, 2021

fees: MultiAsset,

This is v1::MultiAsset which is no compatible with v0::MultiAsset.

On Karura 1011 (The current live runtime), we are using v0::MultiAsset::All and it failed to be decoded on Kusama 9100.

https://github.com/open-web3-stack/open-runtime-module-library/blob/4bd9eed87814e74aa94720f38af0b7328aed2771/xtokens/src/lib.rs#L217

@KiChjang
Copy link
Contributor

IIRC we specifically did not want to allow MultiAsset::All to be used in the fees field in BuyExecution, the justification here being that it is a potential hazard for XCM executors to use up all the transferred assets for fee payment. Thus, starting from V1, we need to always specify the exact amount of assets that we are willing to set aside just for fees, so that we limit the amount of spending that can arise from XCM execution.

@gavofyork
Copy link
Member

See #2815 for information on incompatibilities and #3578 for the fix in Polkadot.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants