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.

Sign up for GitHub

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

Jump to bottom

Investigate whether AllowUnauthenticatedIdentities on AWS::Cognito::IdentityPool should ever be false #68

Closed
ghost opened this issue Oct 18, 2017 · 1 comment
Closed

Investigate whether AllowUnauthenticatedIdentities on AWS::Cognito::IdentityPool should ever be false #68

ghost opened this issue Oct 18, 2017 · 1 comment
Assignees
@tmcelhattan
Labels
feature
Projects
cfn_nag

Comments

@ghost
Copy link

ghost commented Oct 18, 2017

Make a rule to warn about it being true, or figure out if this should be a failing violation
@ghost
Copy link
Author

ghost commented May 8, 2019

https://www.pivotaltracker.com/story/show/164842079

@ghost ghost closed this as completed May 8, 2019
@ghost ghost reopened this May 30, 2019
@ghost ghost added this to To do in cfn_nag Jan 6, 2020
@ghost ghost assigned tmcelhattan Feb 3, 2020
@ghost ghost moved this from To do to In progress in cfn_nag Feb 3, 2020
@tmcelhattan tmcelhattan mentioned this issue Feb 13, 2020
Merged
ghost pushed a commit that referenced this issue Feb 17, 2020
#68 Adding warning rule for cognito IdentityPool AllowUnauthenticated…
a0ed19c 
…Identities - #68 (#372)

* Adding warning rule for cognito AllowUnauthenticatedIdentities

* Refactored rule. also added a util script and a method for checking if a property is referencing a parameter and another method for getting that parameter's Default value.

* cleaning up

* fix error
@ghost ghost moved this from In progress to Done in cfn_nag Feb 17, 2020
@pshelby pshelby closed this as completed Feb 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmcelhattan tmcelhattan

Labels
feature
Projects
cfn_nag
  
Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pshelby @twellspring @tmcelhattan