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

Adding support for multiple AWS secrets to be connected #12

Merged
merged 9 commits into from
Oct 17, 2020

Conversation

MaxGoo
Copy link
Contributor

@MaxGoo MaxGoo commented Oct 17, 2020

Changing AWS_SECRETS_MANAGER_NAME to either be a string or a list of strings.

If a list of strings is provided it will connect each AWS Secrets bucket and merge all the configs from each one into a single config object. The list is processed in order, meaning buckets including later in the list take priority of previous buckets in the case where a config exists in both buckets.

@codecov
Copy link

codecov bot commented Oct 17, 2020

Codecov Report

Merging #12 into master will increase coverage by 1.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   90.00%   91.00%   +1.00%     
==========================================
  Files          11       12       +1     
  Lines          90      100      +10     
  Branches       17       17              
==========================================
+ Hits           81       91      +10     
  Misses          9        9              
Impacted Files Coverage Δ
src/get-secret.ts 33.33% <100.00%> (ø)
src/index.ts 96.00% <100.00%> (+0.54%) ⬆️
test/fixtures/multiple-secrets/config.default.ts 100.00% <100.00%> (ø)

README.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@MaxGoo MaxGoo requested a review from iansu October 17, 2020 21:59
CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Ian Sutherland <ian@iansutherland.ca>
@MaxGoo MaxGoo merged commit edded83 into master Oct 17, 2020
@MaxGoo MaxGoo deleted the allow-multiple-secrets branch October 17, 2020 23:34
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.

2 participants