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

Splice: Stricter interop errors #8031

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ddustin
Copy link
Collaborator

@ddustin ddustin commented Jan 23, 2025

Splice: Stricter interop errors

Error when our peer tries to use the shared output txid as well as the prevtx bytes.

Error when our peer tries to use the shared output txid as well as the prevtx bytes.

Changelog-Changed: Stricter tests for interop with Eclair
Copy link
Contributor

@remyers remyers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a sensible fail safe that also matches what Eclair checks.

@@ -225,7 +225,7 @@ static char *send_next(const tal_t *ctx,

wally_psbt_input_get_outpoint(&in->input, &point);

/* If this the shared channel input, we send funding txid in
/* If this is the shared channel input, we send funding txid in
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: remove last 'in' or redundant one on the next line

@endothermicdev
Copy link
Collaborator

ACK 4eeb5df

@ddustin ddustin force-pushed the ddustin/splice_interop_stricter branch from 4370af4 to 4eeb5df Compare February 1, 2025 15:58
@ddustin ddustin added this to the v25.02 milestone Feb 1, 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