-
Notifications
You must be signed in to change notification settings - Fork 15
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
Investigate what happens when an unkown assets is sent over XCM #416
Comments
So after a quick search, I do not think there is currently a barrier that checks on the assets being sent to us. What happens more or less like this:
|
@gianfra-t thanks for the great ticket description and documentation of your findings 🙏
True, it should eventually fail because we are using the default implementation which returns errors, see here.
I agree, this seems to be the place where the execution of the XCM message would stop in the setup using the asset registry and |
Yes! That is a good question. I think it may be difficult to create a barrier that accounts for the many instructions that may have an unsupported asset on them, but that would be in the general sense. If we are talking only about a Although, I cannot see the immediate benefit other than just save one deposit instruction execution. |
I think I'm fine if we keep it as is and don't check for this in the barrier. I don't see other parachains doing so either. Astar doesn't check for assets, Moonbeam doesn't, Interlay doesn't (though they have an additional check for some This doesn't mean that we can't do it, it's just an indicator that it doesn't seem so be necessary. What's your opinion here @TorstenStueber? |
Thanks @gianfra-t for all the details documented. I'm also fine with keeping everything as is since I think the benefit of one less instruction execution is not worth changing the |
Hey team! Please add your planning poker estimate with Zenhub @b-yap @bogdanS98 @ebma @gianfra-t @TorstenStueber |
Thanks for the research. I agree that we don't need to put more effort into this then. I am fine with closing this ticket @gianfra-t. |
This issue is just a placeholder for the discussion about the topic, and possible action points to take.
Upon receiving an XCM message for asset deposit, we would like to find out if any of the existing barriers block the deposit of assets that are not handled by the chain. We may define an asset "handled" by the chain by those which implement the following:
CurrencyId
representation from it's XCM one, and vice-versa.If there is no such barrier that blocks the execution of these messages, then we should define if it is relevant to add a custom one.
The text was updated successfully, but these errors were encountered: