Skip to content

199 verify signature#201

Merged
kmroz merged 4 commits intomainfrom
199-verify-signature
Aug 28, 2023
Merged

199 verify signature#201
kmroz merged 4 commits intomainfrom
199-verify-signature

Conversation

@bartossh
Copy link
Copy Markdown
Owner

@bartossh bartossh commented Aug 26, 2023

  • Signature can verify itself against the public key.
  • Signature precalculates digest itself again to check if the message is not corrupted before validating the signature.
  • I've introduced new functions to get the public key and private key in the form of EVP_PKEY (from openssl/evp.h) for verification. This will allow for future development when transactions will have a bytes buffer of raw public key encoded to base58. So we know that the issuer with the given address is the owner of a signed message.

Please test on your machine with make memcheck <- to verify against memory leaks.
Test cleanup with make clean.

@bartossh bartossh added enhancement New feature or request feature Feature with business value. labels Aug 26, 2023
@bartossh bartossh self-assigned this Aug 26, 2023
@bartossh bartossh linked an issue Aug 26, 2023 that may be closed by this pull request
Copy link
Copy Markdown
Collaborator

@kmroz kmroz left a comment

Choose a reason for hiding this comment

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

👍🏻

@kmroz kmroz merged commit 9d8ecae into main Aug 28, 2023
@kmroz kmroz deleted the 199-verify-signature branch August 28, 2023 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feature Feature with business value.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verify signature.

2 participants