AllowTopLevelPaidExecutionFrom barrier prevents multi currency transfers which also effects assethub.
#1638
Closed
2 tasks done
Labels
I2-bug
The node fails to follow expected behavior.
I10-unconfirmed
Issue might be valid, but it's not yet known.
T6-XCM
This PR/Issue is related to XCM.
Comments
rimbi
added
I10-unconfirmed
|
What was the motivation for limiting the |
|
To my understanding, it is indeed a security risk to allow unbounded asset withdrawal. But limit it to 1 is just too much, especially when the number is non configurable. |
This was referenced Sep 24, 2023
|
@franciscoaguirre @KiChjang please look into this ASAP. |
bkchr
pushed a commit
that referenced
this issue
Oct 17, 2023
Partially addresses #1638 Still need a better solution to allow devs to have better control of this. --------- Co-authored-by: Keith Yeung <kungfukeith11@gmail.com> Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>
noandrea
pushed a commit
to moonbeam-foundation/polkadot-sdk
that referenced
this issue
Oct 18, 2023
Partially addresses paritytech#1638 Still need a better solution to allow devs to have better control of this. --------- Co-authored-by: Keith Yeung <kungfukeith11@gmail.com> Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>
tdimitrov
pushed a commit
that referenced
this issue
Oct 23, 2023
Partially addresses #1638 Still need a better solution to allow devs to have better control of this. --------- Co-authored-by: Keith Yeung <kungfukeith11@gmail.com> Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>
|
This issue has been mentioned on Polkadot Forum. There might be relevant details there: https://forum.polkadot.network/t/polkadot-release-analysis-v1-3-0/4614/1 |
13 tasks
|
This was fixed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue?
Experiencing problems? Have you tried our Stack Exchange first?
Description of bug
After this change,
AllowTopLevelPaidExecutionFromprevents multi currency transfers which also effectsassethub; multi currency transfers are not possible anymore.Here, also, is a test case in the orml repo which failed after upgrading dependencies to
polkadot-v1.1.0(which includes this change) and this is the solution for the problem, which is the exact copy of the old version of the barrier.Steps to reproduce
No response
The text was updated successfully, but these errors were encountered: