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 support for Microsoft Key Vault HSM key types #17

Open
wants to merge 1 commit into
base: v3
Choose a base branch
from

Conversation

maraino
Copy link

@maraino maraino commented Jan 11, 2023

Description

HSM-protected keys on Microsoft Key Vault have the suffix "-HSM" in the kty property. This commit adds support for unmarshaling those keys.

See https://learn.microsoft.com/en-us/azure/key-vault/keys/about-keys#hsm-protected-keys

HSM protected keys on Microsoft Key Vault have the suffix "-HSM" in the
kty property. This commit adds support for unmarshaling those keys.

See https://learn.microsoft.com/en-us/azure/key-vault/keys/about-keys#hsm-protected-keys
@maraino
Copy link
Author

maraino commented Jan 31, 2023

@csstaub is me again, from square/go-jose#224 and square/go-jose#190

Do you think that this PR makes sense in go-jose? Microsoft is not very kind with keeping standards ;)

@mcpherrinm
Copy link
Collaborator

I'm not really sure if we should support this. On the one hand, it doesn't seem unreasonable, but it's also completely nonstandard

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

2 participants