Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: eliminate crypto.SignerOpts from Sign/Verify #2636

Closed
wants to merge 1 commit into from
Closed

feat: eliminate crypto.SignerOpts from Sign/Verify #2636

wants to merge 1 commit into from

Commits on Aug 7, 2024

  1. feat: eliminate crypto.SignerOpts from Sign/Verify 

    Eliminate io.Reader and crypto.SignerOpts arguments from the
Signator/Verifier interfaces to avoid the need to pass crypto.hash(0) in
multiple places.

For symmetry with httpsignature.Ed25519PubKey add Ed25519PrivKey wrapper
for ed25519.PrivateKey that implements the updated Verifier interface.
In addition the wrapper provides Public() and PublicHex() methods to
simplify the common operations.

Close #2635
    @ibukanov
    ibukanov committed Aug 7, 2024
    Configuration menu
    Copy the full SHA
    6ae3ae3 View commit details
    Browse the repository at this point in the history