c79b3de4d6
payalgo: Report reason to delay before clearing try memory.
We allocate the reason to delay, if any, from the `pay->try_parent`.
So we should not clear the `pay->try_parent` until after we print
the reason.
7 years ago
774af5f817
payalgo: Describe `maxdelay` argument of `pay`.
7 years ago
11ca729d85
wallet, payalgo: Save detail of payment failures for later reporting. ( #1345 )
Pointless for remote failures as those are never sent by
the erring node, but for local failures we can give more
detail.
7 years ago
86290b54d4
routing: Use 64-bit msatoshi for messages to and from routing.
Internally both payment and routing use 64-bit, but the interface
between them used 32-bit.
Since both components already support 64-bit we should use that.
7 years ago
5a267eb831
pay, payalgo: Show erring_node as compressed DER pubkey.
For consistency with other node pubkeys.
7 years ago
26f7014813
payalgo: Add maximum delay.
Fixes : #1086
7 years ago
c1a43a04af
payalgo: Be willing to overpay up to maxfeepercent, for privacy.
This obscures how far an intermediate hop is from the ultimate
payee, and also obscures slightly the exact payment value.
Fixes : #1089
7 years ago
bc5fc692d1
pay: Let `sendpay` modify the recorded `msatoshi` of payments.
7 years ago
7e9750ffee
Reduce variable scopes
7 years ago
9efe123a0d
lightningd/json: Move json helpers specific for lightningd to new module.
7 years ago
881eb80f6a
payalgo: Make 'pay' return similar to 'listpayments'
7 years ago
d181ecbeea
payalgo: Implement retry_for for pay command.
7 years ago
981af478f1
payalgo: Keep and report route and route failures.
7 years ago
0a6e3d1e13
utils: remove tal_tmpctx altogether, use global.
In particular, we now only free tmpctx at the end of main().
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
7 years ago
a0c2686ebd
pay: Have sendpay wait for payment to be saved.
The payment should be stored in a "timely" manner, i.e.
within 10ms.
7 years ago
1e4adb0359
pay: Make sendpay nonblocking.
7 years ago
b857b2e843
Add assertions in various places to ensure tal_fmt doesn't receive NULL as argument for strings.
7 years ago
8e8d7c2aba
pay/sendpay: Use spec names for rhash and r.
7 years ago
1f6008689d
payalgo: Throttle pay command if failure is due to blockheight disagreement.
7 years ago
73cda2f2ae
payalgo: Report route, and result of trying route.
7 years ago
21cfec816d
payalgo: Report number of tries on failure.
7 years ago
6c9d81ef42
payalgo: Remove reporting PAY_TRY_ANOTHER_ROUTE and PAY_UNPARSEABLE_ONION.
These error codes will cause `pay` to retry, so `pay` will never
actually report those error codes.
Those error codes will only get reported at the `sendpay` level.
7 years ago
cf3f19524e
gossip: formalize passing of siphash_seed.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
7 years ago
d23650d2ed
Explain fuzz initial values.
7 years ago
6767434ea9
routing: Use siphash24 for route randomization
Primary idea by @rustyrussell
7 years ago
61d0b50c54
payalgo: Randomize paths as long as we respect maxfeepercent.
7 years ago
9693843141
payalgo: Change sendpay_parent to try_parent.
Now try_parent is used as the parent for all allocations needed
for a try.
7 years ago
216c52940d
gossipd: Add fuzz and seed to getroute request.
7 years ago
91a9c2923f
Mark intentionally unused parameters as such (with "UNUSED")
7 years ago
3dbace3421
Remove redundant casts to same type
7 years ago
e92b710406
tools/generate-wire.py: remove length argument from fromwire_ routines.
We always hand in "NULL" (which means use tal_len on the msg), except
for two places which do that manually for no good reason.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
7 years ago
f351417905
Add assertions to clarify our assumptions about msg != NULL
7 years ago
38535fc36c
payalgo: Create a new failure for paying expired invoice.
7 years ago
4ad1021c2c
payalgo: Repeat pay command if possible.
Fixes : #863
7 years ago
fda26bdcda
payalgo: New file for pay command.
7 years ago
ZmnSCPxj
practicalswift
Rusty Russell
Corné Plooy