eclair-mobile icon indicating copy to clipboard operation
eclair-mobile copied to clipboard

More options for fees - i.e "Cap at 20%"

Open dpad85 opened this issue 6 years ago • 4 comments

Issue by CubicEarth Friday Apr 06, 2018 at 15:15 GMT Originally opened as https://github.com/ACINQ/eclair/issues/548


The only option currently is to cap fees at 3%, or to not enforce at all. I am sure that a more sophisticated approach will be developed in time, but until then, it would be good to have an option to allow fees to be up to some higher amount, while still caping them.

Is it possible that an uncapped fee could be more than 100%? For the next few months, people are going to be making many small 'test' payments to try out this new system. And fees are often higher than 3%. But an 'uncapped' fee seems like it opens up an attack vector.

So perhaps a second toggle button to allow fees up to 20%, or a simple slider to select a fee level?

dpad85 avatar Apr 13 '18 09:04 dpad85

Comment by dpad85 Thursday Apr 12, 2018 at 18:47 GMT


Are you thinking about Eclair Wallet for this? For eclair node, this value can be changed by the user in eclair.conf and I don't think it need a specific UI widget.

For eclair wallet, however, it could certainly be useful.

dpad85 avatar Apr 13 '18 09:04 dpad85

Comment by CubicEarth Thursday Apr 12, 2018 at 22:39 GMT


@dpad85 - You are right, I meant that this issue be attached to the Eclair Wallet repo.

Is there a way to 'move' the issue over? Or should I just close it here, and repost it over there?

dpad85 avatar Apr 13 '18 09:04 dpad85

Ideally - app would say something like: "Found a path to pay an invoice but it requires high fee of: xxx sat which is YYY% of the the invoice value. Do you want to proceed with the payment?"

I know that this introduces a race condition (user clicking OK, but fees changing) - but might be good enough.

irekzielinski avatar Apr 29 '18 20:04 irekzielinski

The fee protection mechanism has been improved with the v0.3.19 release (see https://github.com/ACINQ/eclair-mobile/issues/142 and https://github.com/ACINQ/eclair/pull/844).

This is not as sophisticated as what @irekzielinski proposes though, so I let this issue open for now.

dpad85 avatar Feb 19 '19 15:02 dpad85