You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When we use utility.batch and have set_currency as first extrinsic, then we don't do any validation for the specified currrency.
It can lead to problem like:
Eg. the old DAI token with id 2. There seems be still some of it in our chain ( at least issuance shows something )
if you try to set currency to 2 - it would fail, because it is not accepted
however, if you use set currency as first tx in batch, and set it to 2 -> this would work, because there is old old price from oracle ( from block 2,188,027 ).
so you could pay with this asset.
We could use the same check we have in set currency extrinsic, so check if it is in accepted currencies or has XYK DOT pool. Probably, we should use the same function.
The text was updated successfully, but these errors were encountered:
Description
When we use utility.batch and have
set_currency
as first extrinsic, then we don't do any validation for the specified currrency.It can lead to problem like:
Problematic code:
hydration-node/pallets/transaction-multi-payment/src/lib.rs
Line 655 in c048260
Solution
We could use the same check we have in set currency extrinsic, so check if it is in accepted currencies or has XYK DOT pool. Probably, we should use the same function.
The text was updated successfully, but these errors were encountered: