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 fake signer that implements KMS interface #361
Conversation
cc @bobcallaway |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, and it would be useful when adding some tests in cosigned
This allows us to test code that uses kms.Get by importing the fake KMS package. Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
Need to make one quick change before this is submitted, I want to provide a way to pass in a private key to the loader. Otherwise if you call kms.Get() repeatedly, you'll get back different key values each time. |
Added: A bit of a weird way to do it, but I'm passing a marshalled ECDSA private key via the string ref. If the string ref after |
You know what, it'd be far easier to just pass the private key in the context. I'll clean this up. |
60cee30
to
f0d31ea
Compare
Ready for review! |
The private key is passed in the context. Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
Signed-off-by: Jake Sanders <jsand@google.com>
This allows us to test code that uses kms.Get by importing the fake KMS
package.
Signed-off-by: Hayden Blauzvern hblauzvern@google.com
Summary
Ticket Link
Fixes
Release Note