Skip to content

BIP341: add aux_rand argument to taproot_sign_key#1379

Merged
kallewoof merged 1 commit intobitcoin:masterfrom
DariusParvin:bip341-aux_rand
Jan 4, 2023
Merged

BIP341: add aux_rand argument to taproot_sign_key#1379
kallewoof merged 1 commit intobitcoin:masterfrom
DariusParvin:bip341-aux_rand

Conversation

@DariusParvin
Copy link
Copy Markdown
Contributor

@DariusParvin DariusParvin commented Oct 9, 2022

The schnorr_sign function from the bip340 reference code has a third required argument aux_rand: bytes.

Change:

  • adding aux_rand as an optional argument that defaults to 0x0000...
  • adding bip340_aux_rand as a required argument to the function taproot_sign_key

Copy link
Copy Markdown
Contributor

@jonasnick jonasnick left a comment

Choose a reason for hiding this comment

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

ACK e24f685

Thanks @DariusParvin !

@sipa
Copy link
Copy Markdown
Member

sipa commented Nov 10, 2022

ACK e24f685

@kallewoof kallewoof merged commit 79bb53d into bitcoin:master Jan 4, 2023
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.

4 participants