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
Support aws IAM authentication #50
Conversation
80914c7
to
9f79fe0
Compare
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.
One minor change to the key names, otherwise this looks pretty good.
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.
🙆♂️ Thanks for the addition!
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.
This looks good, thank you for the addition and nice tests covering it!
Co-authored-by: Greg Look <greg@greg-look.net>
Codecov Report
@@ Coverage Diff @@
## master #50 +/- ##
==========================================
+ Coverage 53.82% 55.44% +1.61%
==========================================
Files 10 10
Lines 654 680 +26
Branches 28 29 +1
==========================================
+ Hits 352 377 +25
Misses 274 274
- Partials 28 29 +1
Continue to review full report at Codecov.
|
To sanity check on this vault does expect the iam payload values to be b64 encoded, at the moment I have left that to the consumer to understand as I didn't want to pull additional dependencies into the project. Let me know if you think that is worth changing.