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

switch to globbed elasticache access to reduce policy size #3

Closed
wants to merge 1 commit into from

Conversation

mattbucci
Copy link

@mattbucci mattbucci commented Mar 29, 2019

Closes #2 using the solution proposed in mu gitter by @jeffb4

Potential issues were stated as that cfn-nag needed to be more friendly with globs in Feb 2019 on gitter chat

@mneil
Copy link

mneil commented Mar 29, 2019

Thanks Matt, yes, cfn nag would error this out if we accepted it without adding in an exception for it to pass. This is a good temporary solution. But long-term I think we need to revisit the size of the yaml files on the mu side to reduce it so that extensions like these are more easily added. I'm not going to merge, but it's good reference if anyone else wants to relax these permissions.

@mneil mneil closed this Mar 29, 2019
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