Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: mint and transfer funds back to escrow account on timeout or ack error #172

Merged
merged 3 commits into from
Mar 19, 2024

Conversation

jtieri
Copy link
Member

@jtieri jtieri commented Mar 19, 2024

Previously we were failing to mint and transfer funds back to the escrow account in the case of a timeout or acknowledgement error but were still issuing the refund back to the sending chain. For forwards that involved IBC assets that were wrapped two or more times, this would result in the escrow account balance being less than the total supply of tokens on the counterparty.

@jtieri jtieri added the packet-forward-middleware Label for items related to the packet forward middleware label Mar 19, 2024
@jtieri jtieri enabled auto-merge (squash) March 19, 2024 15:55
@jtieri jtieri requested review from agouin and Reecepbcups March 19, 2024 15:57
@jtieri jtieri merged commit 8e503d4 into release/v6 Mar 19, 2024
4 checks passed
@jtieri jtieri deleted the justin/fix-deps-v6 branch March 19, 2024 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packet-forward-middleware Label for items related to the packet forward middleware
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants