cdklabs/cdk-ecr-deployment

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

Closed this issue · 3 comments

Hi-Fi commented

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

@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 commented

@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?

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