-
Notifications
You must be signed in to change notification settings - Fork 268
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
Auto probing of local channels #762
Conversation
This is a simple mechanism to test our direct peers by sending fake small payments to them. A probe is considered successful if the peer replies with an `UnknownPaymentHash` error.
While at it, what do you think about ACINQ/eclair-mobile#94? |
But the point here is to close zombie channels right? |
To spot them, yes. What happens next may differ depending on whether this happens on server side or mobile app. In a letter case user may be interested to keep the channel open and let peer fix the bug. |
I'm not sure this probing should be enabled on eclair-wallet, it makes more sense for always-on nodes |
I think this feature could be merged, but disabled by default because:
|
This is a simple mechanism to test our direct peers by sending fake
small payments to them. A probe is considered successful if the peer
replies with an
UnknownPaymentHash
error.