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.

Sign up for GitHub

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

Jump to bottom

Lower minimum remote dust limit #1900

Merged
merged 4 commits into from
Sep 29, 2021
Merged

Lower minimum remote dust limit #1900

merged 4 commits into from
Sep 29, 2021

Commits on Sep 28, 2021

  1. Lower minimum remote dust limit 

    We are slowly dropping support for non-segwit outputs, as proposed in
lightning/bolts#894

We can thus safely allow dust limits all the way down to 354 satoshis.
    @t-bast
    t-bast committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    7f2e484 View commit details
    Browse the repository at this point in the history

  2. Check closing tx dust relay 

    In very rare cases where dust_limit_satoshis is negotiated to a low value,
our peer may generate closing txs that will not correctly relay on the
bitcoin network due to dust relay policies.

When that happens, we detect it and force-close instead of completing the
mutual close flow.
    @t-bast
    t-bast committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    0b9f28a View commit details
    Browse the repository at this point in the history

  3. Disallow non-segwit scripts in close API 

    We shouldn't use non-segwit scripts anymore as they can in theory mess
with the dust limits (and we should encourage migration to segwit).
    @t-bast
    t-bast committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    24350fa View commit details
    Browse the repository at this point in the history

  4. Add release notes

    @t-bast
    t-bast committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    ca2c87f View commit details
    Browse the repository at this point in the history