Skip to content
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

Fee calculations incompatibility with LND result in unilaterally closed channels #1351

Closed
azhmur opened this issue Apr 9, 2018 · 4 comments

Comments

@azhmur
Copy link

azhmur commented Apr 9, 2018

Steps to reproduce:

  1. lnd open channel to clighting
  2. after some blocks lnd update fee
  3. clighting consider this fee less than min_fee and unilaterally close the channel.

In my exact case LND proposed 250 and clighting has range 253-6785:
lightningnetwork/lnd#1058

u32 feerate_min(struct lightningd *ld)

PS I don't sure who is wrong side in this case. But it is impossible to maintain channels between lnd and clighting right now.

@ZmnSCPxj
Copy link
Contributor

ZmnSCPxj commented Apr 9, 2018

This seems a consequence of @rustyrussell's infamous omg-bitcoind-vbytes-must-die-die-die PR: #1251. We hardcode a minimum feerate of 253 even if bitcoind gives us 250, because we calculate fees more sanely than bitcoind does.

@LightningK0ala
Copy link

I'm still having a lot of issues with this.

@niftynei
Copy link
Contributor

niftynei commented Jan 7, 2019

per a convo on IRC today, it looks like lnd has a bug with update_fee which may be related

roasbeef: lnd has a fixed for an update_fee related bug, might explain some of the random force closes even between ourselves, should land in this week and we'll follow up with a minor release to get ppl on the network updated

@darosior
Copy link
Contributor

darosior commented Nov 4, 2019

I think it has finally been fixed by #3215, @azhmur maybe we can now close the issue ?

@azhmur azhmur closed this as completed Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants