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

Enhancement | Enable default-NACL and Disable dedicated-NACL by default #37

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

rodriguez-matias
Copy link
Contributor

What?

  • Ensure that restrictive Network ACLs (NACLs) are disabled by default.
  • In the default configuration, Dedicated-NACLs will be disabled, variables setup:
  manage_default_network_acl    = true
  public_dedicated_network_acl  = false // use dedicated network ACL for the public subnets.
  private_dedicated_network_acl = false // use dedicated network ACL for the private subnets.

Why?

  • Prevent potential complications, streamline user experience, and ensure that the use of NACLs is only when explicitly required and approved.
  • Provide a safer and more user-friendly default setting by having restrictive NACLs disabled.

References

@rodriguez-matias rodriguez-matias added enhancement New feature or request patch labels Nov 1, 2023
@rodriguez-matias rodriguez-matias self-assigned this Nov 1, 2023
@rodriguez-matias rodriguez-matias requested a review from a team as a code owner November 1, 2023 13:27
@rodriguez-matias rodriguez-matias merged commit f21d736 into master Nov 6, 2023
4 checks passed
@rodriguez-matias rodriguez-matias deleted the ISSUE-36/feat/defaul-network-acl branch November 6, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants