Skip to content

secp256k1/ecdsa: Consistent sig recovery errors.#2914

Merged
davecgh merged 1 commit intodecred:masterfrom
davecgh:secp256k1_consistent_compact_sig_errs
Apr 7, 2022
Merged

secp256k1/ecdsa: Consistent sig recovery errors.#2914
davecgh merged 1 commit intodecred:masterfrom
davecgh:secp256k1_consistent_compact_sig_errs

Conversation

@davecgh
Copy link
Copy Markdown
Member

@davecgh davecgh commented Apr 3, 2022

This converts the error handling in compact signature recovery to be consistent with the rest of the code base such that it returns concrete errors that support errors.Is/As and thus are programmatically detectable by callers.

@davecgh davecgh added this to the 1.8.0 milestone Apr 3, 2022
Copy link
Copy Markdown
Member

@dnldd dnldd 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 besides minor typo.

Comment thread dcrec/secp256k1/ecdsa/error.go Outdated
@davecgh davecgh force-pushed the secp256k1_consistent_compact_sig_errs branch from a4f479c to a5b3e5d Compare April 5, 2022 22:33
This converts the error handling in compact signature recovery to be
consistent with the rest of the code base such that it returns concrete
errors that support errors.Is/As and thus are programmatically
detectable by callers.
@davecgh davecgh force-pushed the secp256k1_consistent_compact_sig_errs branch from a5b3e5d to d205e99 Compare April 7, 2022 15:42
@davecgh davecgh merged commit d205e99 into decred:master Apr 7, 2022
@davecgh davecgh deleted the secp256k1_consistent_compact_sig_errs branch April 7, 2022 16:06
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.

5 participants