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

(feat) add support to secrets (SSM and Secrets Manager) #143

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

hectorgrecco
Copy link

@hectorgrecco hectorgrecco commented Dec 10, 2021

Issue #, if available:
We currently have support for fixed environment variables, but not for sensitive data store at SSM or Secrets Manager.

Description of changes:
I've added a parameter named secrets where you can set SECRET_NAME=AWS_ARN value. I've added a Regex to validate the ARN. I did not write tests, so... If someone could write those tests, the community would be happy :)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@amazreech
Copy link
Contributor

amazreech commented May 10, 2024

Hi @hectorgrecco, thank you so much for your contribution. Apologies on the delay.
We will be working on reviewing Pull Requests on the repository. In the mean time please ensure that below steps, if not already done, are taken care of in your PR:

  1. Verify if PR follows semantic pull request conventions.

  2. Please be sure to run npm run package command to update dist/ folder with latest dependencies.

  3. Resolve merge conflicts on the PR if any.

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.

None yet

2 participants