renepay: refactor and bugfixes #8798
Open
+269
−742
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Lagrang3
force-pushed
the
renepay_sendpay_race
branch
from
1ecdf9f to
da46e19
Compare
We remove the auxiliary RPC renesenday (only used internally) in favor of a function that builds the onion, stores the shared secrets needed to recover the onion reply, and calls sendonion/injectpaymentonion. This solves a concurrency race ``` **BROKEN** plugin-cln-renepay: Unable to parse sendpay_failure ``` in which we are waiting for renesenday to return in order to record the shared secrets but we get the a sendpay_failure notification with an onionreply before we have secrets to decode it. It also solves a missing JSON id seen in the logs ``` DEBUG plugin-cln-renepay: JSON reply with unknown id ``` because renesendpay was using his command variable to issue an RPC to sendonion/injecpaymentonion and would fail or succeed the command before those RPCs were done. This also meant that the callback functions were silently being ignored. Changelog-Fixed: renepay: fixes a race condition that leads to **BROKEN** plugin-cln-renepay: Unable to parse sendpay_failure Signed-off-by: Lagrang3 <[email protected]>
Lagrang3
force-pushed
the
renepay_sendpay_race
branch
from
da46e19 to
eb5fd83
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We remove the auxiliary RPC
renesendpay(only used internally) in favor of a function that builds the onion, stores the shared secrets needed to recover the onion reply, and callssendonion/injectpaymentonion.This solves a concurrency race
in which we are waiting for
renesendpayto return in order to record the shared secrets but we get the asendpay_failurenotification with an onionreply before we have secrets to decode it.It also solves a missing JSON id seen in the logs
because
renesendpaywas using his command variable to issue an RPC tosendonion/injecpaymentonionand would fail or succeed the command before those RPCs were done. This also meant that the callback functions were silently being ignored.Solves issues #8789 and #8345.