Skip to content

Fix BCH Schnorr signatures throwing BAD_SIGNATURES error#3568

Merged
nitsujlangston merged 6 commits intobitpay:masterfrom
kajoseph:fix/bchBadSignatures
Apr 4, 2023
Merged

Fix BCH Schnorr signatures throwing BAD_SIGNATURES error#3568
nitsujlangston merged 6 commits intobitpay:masterfrom
kajoseph:fix/bchBadSignatures

Conversation

@kajoseph
Copy link
Copy Markdown
Collaborator

@kajoseph kajoseph commented Apr 3, 2023

No description provided.

}
);
signatures = signatures.flat().sort((a, b) => a.inputIndex - b.inputIndex);
// DEBUG
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You leaving this here for future convenience or did you forget to remove?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Left it there for debugging

Copy link
Copy Markdown
Contributor

@escottalexander escottalexander left a comment

Choose a reason for hiding this comment

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

Looks good. Pipeline passes for me locally so might just need rerunning.

@nitsujlangston nitsujlangston merged commit 45e3983 into bitpay:master Apr 4, 2023
@nitsujlangston nitsujlangston temporarily deployed to github-pages April 4, 2023 13:18 — with GitHub Pages Inactive
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