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

New Published Rules - aws-network-acl-allows-public-ingress #2874

Merged

Conversation

semgrep-dev-pr-bot[bot]
Copy link
Contributor

Wahoo! New published rules with aws-network-acl-allows-public-ingress from @gabriellesc.

See semgrep.dev/s/u21:aws-network-acl-allows-public-ingress for more details.

Thanks for your contribution! ❤️

@gabriellesc
Copy link
Contributor

This PR makes some important changes to this rule: #2876
(I would have closed this PR and opened a new one, but I don't have permission to do that)

…ault_network_acl (#2876)

* wip: move rule files

* feat: add contextual comments to terraform.aws.security.aws-network-acl-allows-public-ingress rule and support aws_default_network_acl
@enncoded enncoded closed this May 1, 2023
@enncoded enncoded reopened this May 1, 2023
@enncoded enncoded merged commit ca6416a into develop May 1, 2023
@enncoded enncoded deleted the semgrep-live-u21-aws-network-acl-allows-public-ingress branch May 1, 2023 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants