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.

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

Add Thumbprint support for opaque signers #38

Merged
merged 1 commit into from Mar 29, 2023

Conversation

maraino
Copy link

@maraino maraino commented Mar 6, 2023

Description

This commit allows getting a thumbprint for the type OpaqueSigner. So if a key is backed by a custom signer, for example, a YubiKey or a cloud signer, we should be able to get the thumbprint for it as long as the public key is one of the supported ones.

This commit allows getting a thumbprint for the type OpaqueSigner. So if
a key is backed by a custom signer, for example, a YubiKey or a cloud
signer, we should be able to get the thumbprint for it as long as the
public key is one of the supported ones.

Signed-off-by: Mariano Cano <mariano.cano@gmail.com>
jwk.go Show resolved Hide resolved
@pgporada
Copy link

Thank you for submitting this PR, sorry that it has lingered for so long. Tell Jenessa hey!

@maraino maraino requested a review from pgporada March 15, 2023 00:26
@mcpherrinm mcpherrinm merged commit 9f88c6f into go-jose:v3 Mar 29, 2023
1 check passed
@maraino maraino deleted the opaque-signer-thumbprint branch March 29, 2023 17:58
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.

None yet

4 participants