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

Dust limit too low when opening channels with latest lnd #7541

Closed
mb300sd opened this issue Oct 20, 2021 · 2 comments · Fixed by #7542
Closed

Dust limit too low when opening channels with latest lnd #7541

mb300sd opened this issue Oct 20, 2021 · 2 comments · Fixed by #7542
Milestone

Comments

@mb300sd
Copy link

mb300sd commented Oct 20, 2021

Is it time to change this to DUST_LIMIT_DEFAULT_SAT_SEGWIT in lnutil.py?

        # technically this could be using the lower DUST_LIMIT_DEFAULT_SAT_SEGWIT
        # but other implementations are checking against this value too; also let's be conservative
        if self.dust_limit_sat < bitcoin.DUST_LIMIT_DEFAULT_SAT_LEGACY:

The latest lnd is using the segwit dust limit now, it might start getting difficult for people to open channels as more nodes upgrade.

lightningnetwork/lnd@ddbd0b6

@bitromortac
Copy link
Contributor

Yes, I'm aware of this, will submit a PR soon, thanks!

@SomberNight
Copy link
Member

Note that the quoted code comment referencing the 294 sat value is incorrect; the lowest value we can safely accept is probably 330 sat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants