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

tapchannel: import funding output proofs for responder during co-op close #987

Merged
merged 4 commits into from
Jul 3, 2024

Conversation

Roasbeef
Copy link
Member

@Roasbeef Roasbeef commented Jul 2, 2024

Otherwise, the transfer insert for the responder may fail as they don't
have the funding output in their database, so shipping the transfer
fails as it can't find the proof.

Fixes #985.

Roasbeef added 4 commits July 2, 2024 17:08
In this commit, as a prep to reuse this routine elsewhere, we decouple
the method into a function.
This version includes an scid field in the close req.
…lose

Otherwise, the transfer insert for the responder may fail as they don't
have the funding output in their database, so shipping the transfer
fails as it can't find the proof.

Fixes #985.
Without this, we'll fail to fetch the proofs for a grouped asset in
`importOutputProofs`.
@Roasbeef Roasbeef force-pushed the responder-coop-proof-import branch from f8d357c to a6e193d Compare July 3, 2024 00:09
@Roasbeef Roasbeef merged commit 02697e2 into main Jul 3, 2024
16 checks passed
@jharveyb
Copy link
Contributor

jharveyb commented Jul 3, 2024

Post-merge LGTM 👍🏽

guggero added a commit to guggero/lightning-terminal that referenced this pull request Jul 8, 2024
Since lightninglabs/taproot-assets#987 was merged, this TODO and if
statement is no longer needed to make the test pass.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[bug]: responder of taproot assets channel doesn't have proof for commitment transaction
2 participants