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

Peers need to check each other's dust limit #894

Merged
merged 4 commits into from Oct 6, 2021
Merged

Peers need to check each other's dust limit #894

merged 4 commits into from Oct 6, 2021

Commits on Sep 15, 2021

  1. Peers need to check each other's dust limit 

    Since HTLCs below this amount will not appear in the commitment tx, they
are effectively converted to miner fees. The peer could use this to grief
you by broadcasting its commitment once it contains a lot of dust HTLCs.

Fixes #696
    @t-bast
    t-bast committed Sep 15, 2021
    Configuration menu
    Copy the full SHA
    a19d5fb View commit details
    Browse the repository at this point in the history

  2. Add network dust thresholds 

    As implemented in Bitcoin Core's default relay policy.
    @t-bast
    t-bast committed Sep 15, 2021
    Configuration menu
    Copy the full SHA
    7e3dce4 View commit details
    Browse the repository at this point in the history

  3. Drop non-segwit support in shutdown 

    This allows dust limit to go as low as 354 sats without creating relay
issues with default node policies.

We add a requirement that dust limit cannot be lower than 354 sats.
This ensures implementers don't have to figure this subtlety on their own.
    @t-bast
    t-bast committed Sep 15, 2021
    Configuration menu
    Copy the full SHA
    9ff1387 View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2021

  1. fixup! Add network dust thresholds

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