Skip to content

Improve origin check for token registrations#16

Merged
yrong merged 1 commit intoyrong:ron/register-token-origin-as-ownerfrom
vgeddes:vincent/register-token-origin-as-owner
Mar 13, 2025
Merged

Improve origin check for token registrations#16
yrong merged 1 commit intoyrong:ron/register-token-origin-as-ownerfrom
vgeddes:vincent/register-token-origin-as-owner

Conversation

@vgeddes
Copy link
Copy Markdown

@vgeddes vgeddes commented Mar 13, 2025

No description provided.

@yrong yrong merged commit 75d4e20 into yrong:ron/register-token-origin-as-owner Mar 13, 2025
yrong added a commit that referenced this pull request Mar 20, 2025
…h#57)

* Allow only the asset owner to register it on BH

* Create origin for local assets

* Add integration tests

* Improve token registration check (#16)

* Assert events on BH

* Cleanup

* Update cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge_v2_outbound_edge_case.rs

Co-authored-by: Vincent Geddes <117534+vgeddes@users.noreply.github.com>

* Move the snowbridge-runtime-common

* Update bridges/snowbridge/runtime/runtime-common/src/register_token.rs

Co-authored-by: Vincent Geddes <117534+vgeddes@users.noreply.github.com>

* Rename

* Update bridges/snowbridge/runtime/runtime-common/src/fee_handler.rs

Co-authored-by: Vincent Geddes <117534+vgeddes@users.noreply.github.com>

* XcmExportFeeToSibling to V1 and AssetOwner to V2

* Update bridges/snowbridge/runtime/runtime-common/src/v2/register_token.rs

Co-authored-by: Adrian Catangiu <adrian@parity.io>

* Update bridges/snowbridge/runtime/runtime-common/src/v2/register_token.rs

Co-authored-by: Adrian Catangiu <adrian@parity.io>

---------

Co-authored-by: Vincent Geddes <117534+vgeddes@users.noreply.github.com>
Co-authored-by: Adrian Catangiu <adrian@parity.io>
yrong pushed a commit that referenced this pull request Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants