Skip to content

Fast transfers: merge into main#329

Merged
karim-en merged 6 commits intomainfrom
audit-fast-transfers
May 12, 2025
Merged

Fast transfers: merge into main#329
karim-en merged 6 commits intomainfrom
audit-fast-transfers

Conversation

@karim-en
Copy link
Copy Markdown
Collaborator

The origin PRs are:
#147
#309
#312
#313
#324

kiseln and others added 5 commits March 6, 2025 15:53
* Implement simple fast transfer

* run fmt

* Fast transfers implementation

* Storage deposit and fee fixes

* Minor code review fixes

* Fix fast transfer to other chain message

* add integration tests

* Fix token refund

* cargo fmt

* Code review fixes

* Fail during fast transfer if transfer already finalised

* Do not burn tokens during failed fast transfer

* Mark fast transfer as completed

* Implement restriction on claiming fee before fast transfer is finalised

* feat: pass the fast relayer over `FastFinTransferMsg` (#265)

* feat: pass the fast relayer over `FastFinTransferMsg`

* Remove `msg` from fast trasnfer relayer

* Revert some changes

* Remove extra clone

* Fix clippy

* Backwards compatibility for old transfer message on Near

* Add test and fix fmt

* Change keccak to sha256 for gas efficiency

* Improve fast transfer tests structure

* Remove fast transfer after it's done and refund storage

* Make fast transfer storage upgradeable

* Use signer_id for paying storage during fast transfer

* fix clippy

---------

Co-authored-by: karim-en <karim.alabtakh@nearone.org>
* feat: add native fee restriction (based on implementation from main)
* fix: fix transfer message storage

* Add state migration with test

* fix the `StorageKey` order

---------

Co-authored-by: kiseln <3428059+kiseln@users.noreply.github.com>
@karim-en karim-en requested a review from a team May 12, 2025 16:12
@karim-en karim-en merged commit 65fee07 into main May 12, 2025
16 of 17 checks passed
@kiseln kiseln deleted the audit-fast-transfers branch June 24, 2025 19:04
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