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

Add and switch to new types for revocation #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

canndrew
Copy link
Member

Add the UInt48, RevocationKey, CommitmentPubKey, CommitmentSeed and
RevocationSet types.

A RevocationKey is a secret key that can be used to revoke a
transaction. It's shared in the revoke_and_ack message and stored in the
user's wallet in a RevocationSet. A CommitmentPubKey is the public key
of a RevocationKey, also known as a "per commitment point". A
CommitmentSeed is the master revocation key for a side of a channel and
can be used to derive all the revocation keys for that side of that
channel.

A RevocationSet stores a set of revocation keys in a compact form
specified in BOLT #3 which allows earlier keys to be derived from later
keys.

Add the UInt48, RevocationKey, CommitmentPubKey, CommitmentSeed and
RevocationSet types.

A RevocationKey is a secret key that can be used to revoke a
transaction. It's shared in the revoke_and_ack message and stored in the
user's wallet in a RevocationSet. A CommitmentPubKey is the public key
of a RevocationKey, also known as a "per commitment point". A
CommitmentSeed is the master revocation key for a side of a channel and
can be used to derive all the revocation keys for that side of that
channel.

A RevocationSet stores a set of revocation keys in a compact form
specified in BOLT joemphilips#3 which allows earlier keys to be derived from later
keys.
@canndrew canndrew force-pushed the nblockchain-master-with-revocation-changes branch from c15a629 to 67aeb07 Compare June 29, 2020 12:37
Allows converting a RevocationSet to/from a list of keys. This allows
serializing/deserializing the set.
makeChannelReestablish was using hard-coded commitment numbers and keys
rather than the info in the Commitments object that it gets passed.

This fix makes it use the values in the Commitments object instead.
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 this pull request may close these issues.

1 participant