Skip to content

Conversation

laszlovandenhoek
Copy link
Contributor

Fixes #7216

Return the public key in SubjectPublicKeyInfo format for both RSA and ECDSA keys, per the AWS documentation: https://docs.aws.amazon.com/cli/latest/reference/kms/get-public-key.html#output

Please refer to the contribution guidelines in the README when submitting PRs.

@localstack-bot
Copy link
Contributor

localstack-bot commented Mar 13, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Copy link
Contributor

@localstack-bot localstack-bot left a comment

Choose a reason for hiding this comment

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

Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining our Slack Community and share your PR on the #community channel to share your contributions with us. Please make sure you are following our contributing guidelines and our Code of Conduct.

@laszlovandenhoek
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

Fixes localstack#7216

Return the public key in SubjectPublicKeyInfo format for both RSA and ECDSA keys, per the AWS documentation: https://docs.aws.amazon.com/cli/latest/reference/kms/get-public-key.html#output
Copy link
Member

@viren-nadkarni viren-nadkarni left a comment

Choose a reason for hiding this comment

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

Checked that other KMS operations using pubkey are unaffected.

LGTM! Thanks for contributing @laszlovandenhoek!

@viren-nadkarni viren-nadkarni merged commit eb15fd7 into localstack:master Mar 15, 2023
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.

bug: unable to verify AWS KMS asymmetric key signatures generated by localstack locally with OpenSSL
3 participants