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

channeld: don't close connection on invalid badonion code (LND bug?) #6425

Merged

Conversation

rustyrussell
Copy link
Contributor

channeld WARNING: Bad update_fail_malformed_htlc failure code 4103

Warren Togami reports this happening with Bitrefill on every reconnect, so it's clearly something LND does.

(4103 is TEMPORARY_CHANNEL_FAILURE, which does not belong in update_fail_malformed_htlc).

Changelog-Changed: Protocol: We allow update_fail_malformed_htlc with invalid error codes (LND?)

```
channeld WARNING: Bad update_fail_malformed_htlc failure code 4103
```

Warren Togami reports this happening with Bitrefill on every reconnect,
so it's clearly something LND does.

(4103 is TEMPORARY_CHANNEL_FAILURE, which does not belong in update_fail_malformed_htlc).

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-Changed: Protocol: We allow update_fail_malformed_htlc with invalid error codes (LND?)
@wtogami
Copy link
Contributor

wtogami commented Jul 22, 2023

2023-07-22T11:14:04.513Z UNUSUAL 03d607f3e69fd032524a867b288216bfab263b6eaee4e07783799a6fe69bb84fac-channeld-chan#1401: Bad update_fail_malformed_htlc failure code 4103

Confirmed this patch cherry-picked upon v23.05.2 recovered the channel that was otherwise disconnecting immediately on every reconnect.

@rustyrussell rustyrussell merged commit b6f2eb9 into ElementsProject:master Jul 24, 2023
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants