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

Issue 1 #4

Merged
merged 3 commits into from Feb 1, 2017
Merged

Issue 1 #4

merged 3 commits into from Feb 1, 2017

Conversation

aceew
Copy link
Owner

@aceew aceew commented Jan 27, 2017

fixes #1

…ises when the nev does not support them. Added ability to decrypt multiple env vars at once using kms
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0dd4bcd on issue-1 into 1aee00d on master.

@@ -54,6 +54,31 @@ export default class LambdaEnvVars {
}

/**
* Decrypts a list of environment variables and returns them in an object where the keys are the
* env variable keys and the names are the decrypted values.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be 'values are the decrypted values'?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Yes

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2c807f0 on issue-1 into 1aee00d on master.

@aceew aceew merged commit e7be02a into master Feb 1, 2017
@aceew aceew deleted the issue-1 branch February 1, 2017 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It should be possible to specify an array of env vars to decrypt
3 participants