Skip to content

Conversation

bhandras
Copy link
Member

@bhandras bhandras commented Jan 11, 2023

This PR adds a new argument to MuSig2CreateSession which passes in which MuSig2 version the client would like to create a session for. We also change how the signers' public keys are passed as depending on the version these may be of different length.

Depends on lightningnetwork/lnd#7171

Pull Request Checklist

  • PR is opened against correct version branch.
  • Version compatibility matrix in the README and minimal required version
    in lnd_services.go are updated.
  • Update macaroon_recipes.go if your PR adds a new method that is called
    differently than the RPC method it invokes.

Copy link
Contributor

@positiveblue positiveblue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯

This commit adds a new argument to MuSig2CreateSession which passes in
which MuSig2 version the client would like to create a session for. We
also change how the signers' public keys are passed as depending on the
version these may be of different length.
Copy link
Contributor

@guggero guggero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@bhandras bhandras merged commit 6d34872 into lightninglabs:lnd-16-0 Feb 6, 2023
@bhandras bhandras deleted the musig-versioning branch February 6, 2023 14:59
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.

3 participants