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

Feature request: Support getting external registry credentials from secrets/parameter store #171

Closed
Hi-Fi opened this issue Feb 9, 2022 · 3 comments · Fixed by #177
Closed
Assignees
Labels
enhancement New feature or request

Comments

@Hi-Fi
Copy link
Contributor

Hi-Fi commented Feb 9, 2022

Currently construct takes username and password as plain text parameter, which then end up plain text to both CloudFormation template and also at every lambda execution to CloudWatch logs.

It would be possible to be able to read those values from secrets manager, and be able to add those in a way that CDK would handle adding the needed policies.

I think setting requires following things:

  • Secret name/ARN
  • key for username
  • key for password

References

@wchaws
Copy link
Contributor

wchaws commented Feb 10, 2022

@Hi-Fi Thanks for your nice request. The feature is meaningful. I will support that in the next release. PR is welcomed as well.

@Hi-Fi
Copy link
Contributor Author

Hi-Fi commented Mar 4, 2022

@wchaws If would have time for creation for PR, how do you see this should be done?

Maybe latter would be clearer and easier to read?

@wchaws
Copy link
Contributor

wchaws commented Mar 6, 2022

@Hi-Fi I would prefer to the 2nd approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants