event icon indicating copy to clipboard operation
event copied to clipboard

Add `maxfeepercent` for C-Lightning `pay` command to permit higher payment success rate

Open willcl-ark opened this issue 3 years ago • 5 comments

I have been having some difficulty paying to Muun wallet from my C-Lightning node, as they appear to have some very high-fee channels on the recipient side at the moment...

signal-2022-02-16-185919_001

This is solvable when interacting directly with my node, by increasing the allowed maxfeepercent parameter on the pay command, e.g. instead of:

lightning-cli pay bolt11

...which will fail with "fee exceeds our fee budget", I can use:

lightning-cli pay -k bolt11=invoice maxfeepercent=0.6

...which increases the fee budget and can therefore find more routes to try.

I think it would be generally useful if we could somehow integrate a maxfeepercent toggle into the UI? Perhaps it could appear only after first payment failure, if it's deemed "too advanced" (or to be even more precise, could appear if the returned failreason contains Fee exceeds our fee budget, or else could be in an "advanced" tab when scanning the invoice?

willcl-ark avatar Feb 17 '22 10:02 willcl-ark

@Bosch-0 let's add this to the v0.6.1 sprint. Definitely need this.

kaloudis avatar Feb 18 '22 05:02 kaloudis

#860

deregs avatar Feb 20 '22 05:02 deregs

Sounds good to me! Looks like it can be apart of 0.6.0 thanks to Deregs <3

Bosch-0 avatar Feb 21 '22 04:02 Bosch-0

This has been added to v0.6.0. Can you test it out @willcl-ark?

kaloudis avatar Mar 02 '22 02:03 kaloudis

Thanks guys. I have tested this on 0.6 today but unfortunately fee situation has improved on Muun wallet node, so I wasn't able to invoke the new behaviour even with a 5 sat payment :(

After the conference I plan to create a dummy node which will have a very high fee channel so that I can force the new mode.

willcl-ark avatar Mar 03 '22 15:03 willcl-ark

I think I can close this for you now, right? let me know if you want it re-opened for some reason.

willcl-ark avatar Dec 20 '22 10:12 willcl-ark