Skip to content

POC for ssm parameter tokens#2

Draft
blackcathacker wants to merge 1 commit intomasterfrom
ssm_param
Draft

POC for ssm parameter tokens#2
blackcathacker wants to merge 1 commit intomasterfrom
ssm_param

Conversation

@blackcathacker
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@joeswiggin joeswiggin left a comment

Choose a reason for hiding this comment

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

Might want to add to the readme something about configuring the new options and how one would find the appropriate values in IAM.

@blackcathacker
Copy link
Copy Markdown
Author

Yeah. @jwiggins78 I might need help with how to find the correct role and such since I'm still a little fuzzy on that. Also I still need to test this. Am curious if you have any feedback on the approach to caching or if there is any way to make it more resilient...

@joeswiggin
Copy link
Copy Markdown

As you test and work against policies that don't allow the assumption, you'll see that you will have uncaught promise exceptions.... maybe a catch on there somewhere will be appropriate to throw a known error type? Otherwise, I think the caching is fine. I'd be happy to pair with you to help describe the requirements of setting up the roles for assumption to work.

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