search

paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK
https://polkadot.network/
1.63k stars 570 forks source link

Checking account for ForeignAssets teleports should not be tracked in Asset Hubs #2753

Open acatangiu opened 6 months ago

acatangiu commented 6 months ago

Thanks for pointing it out! While the CheckedAccount in BH is unit type so could check-out be safely ignored here?

_Originally posted by @yrong in https://github.com/paritytech/polkadot-sdk/pull/2522#discussion_r1430839884_

acatangiu commented 4 months ago

Not tracked in Asset Hubs either: https://github.com/polkadot-fellows/runtimes/blob/59172d5fb8feea2646490618bead3b2b4fb06220/system-parachains/asset-hubs/asset-hub-kusama/src/xcm_config.rs#L101

acatangiu commented 4 months ago

Seems this is intended because the system chains are not meant to recognize the total issuance in the network. For assets, which are created on Asset Hub, it should still maintain the total issuance.

It's now unclear if we should be tracking teleports of foreign assets, because we can really only teleport with the chain/protocol that issued the asset, and we basically trust them to determine their total issuance. The foreign location that issued the asset is its owner and could force change the balance of the checking account anyway.