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

htlcswitch+lnwallet+channeldb: invalid sig fix #3872

Merged
merged 7 commits into from
Jan 23, 2020

Commits on Jan 6, 2020

  1. lnwallet: restore missing field in local log update

    When restoring an htlc fulfill update from disk, the payment hash wasn't
    restored previously.
    joostjager committed Jan 6, 2020
    Configuration menu
    Copy the full SHA
    b3bd470 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    41c207e View commit details
    Browse the repository at this point in the history

Commits on Jan 15, 2020

  1. lnwallet/test: test remote update after restart

    This test asserts that remote updates that are locked-in on the local
    commitment, but haven't been signed for on the remote commitment, are
    properly restored after a restart.
    joostjager committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    6a86689 View commit details
    Browse the repository at this point in the history
  2. channeldb: update UpdateCommitment description

    This method is only used to update the local commitment transaction.
    Updated comment accordingly.
    joostjager committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    ea892fc View commit details
    Browse the repository at this point in the history
  3. channeldb: extract log update serialization

    Extract functionality to methods as a preparation for serializing remote
    log updates.
    joostjager committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    2d37d34 View commit details
    Browse the repository at this point in the history
  4. lnwallet: extract pending local updates restore method

    Extract method in preparation for restoring dangling remote updates. We
    need to get rid of the early return.
    joostjager committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    ed8fa35 View commit details
    Browse the repository at this point in the history
  5. lnwallet: restore unsigned acked remote updates

    This commit updates the channel state machine to
    persistently store remote updates that we have received a
    signature for, but that we haven't yet included in a commit
    signature of our own.
    
    Previously those updates were only stored in memory and
    dropped across restarts. This lead to the production of
    an invalid signature and channel force closure. The remote
    party expects us to include those updates.
    joostjager committed Jan 15, 2020
    Configuration menu
    Copy the full SHA
    8257940 View commit details
    Browse the repository at this point in the history