-
-
Notifications
You must be signed in to change notification settings - Fork 375
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 secretsManager middleware #227
Conversation
…e set in the context - changed places where mockReset is used where it should be mockClear (since we just want to reset the invocation count) - cache the SSM response, and apply the cached params even when skip calling SSM
Codecov Report
@@ Coverage Diff @@
## master #227 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 18 19 +1
Lines 455 492 +37
Branches 93 100 +7
=====================================
+ Hits 455 492 +37
Continue to review full report at Codecov.
|
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.
Seems perfect! Great job :)
Let me version bump and get this one merged and released.
Would you like to submit a PR also to have this as part of version 1.x?
Thanks! Yes, would put a PR for the 1.0.0 branch as well (know the drill by now :-P) |
|
||
### Sample Usage | ||
|
||
Simplest usage, exports parameters as environment variables. |
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.
exports parameters as environment variables.
Copy-paste from SSM middleware?) Looks like no option to export as environment variable in this middleware, only context
object
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.
Excellent addition!
Added middleware for
secretsManager
with corresponding tests and README changes.Addresses #151