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

This module, doesnt integrate with the Network Firewall Module #978

Open
jseiser opened this issue Aug 28, 2023 · 7 comments
Open

This module, doesnt integrate with the Network Firewall Module #978

jseiser opened this issue Aug 28, 2023 · 7 comments
Labels
Milestone

Comments

@jseiser
Copy link

jseiser commented Aug 28, 2023

Is your request related to a problem? Please describe.

Their is now a network firewall module, but it does not integrate with this module, so even when using them both, you are left in a situation where you can not actually route traffic through the firewall in its normal manner.

Ive also created this here: terraform-aws-modules/terraform-aws-network-firewall#1

Since im not sure where it should live, since both modules are impacted

Describe the solution you'd like.

  1. Should be able to create Firewall Subnets
  2. Should be able to control/override routes for the module so once the firewall is deployed, you can update the routing tables to point to it.

How we currently handle this


    Disable public subnets, and nat gateways
    Create Public Subnets and NAT Gateway, and Internet Gateway
    Create the route table for the NAT gateways
    Create 2 Firewall Subnets and their route tables
    Deploy this ( actually your old beta version) module passing in the firewall subnets in the subnet mapping
    Create the routes for the IGW
    Create the routes for the firewall subnets
    Create all the route associations

Additional context

I think the main issue is not being flexible enough in subnet creation, and not being able to override routes from the Firewall Module, since anytime the VPC module is re-ran, they would be removed/correct I believe.

@dimakievukraine
Copy link

I have also issue with AWS Network Firewall module. According to AWS Documentation separate subnets for Network Firewall should be created. Then traffic routes from FW subnet to NAT GW ( public subnet ) and then back from public subnet to FW VPC endpoint. The issue that if I create AWS Network Firewall with 3 endpoints in 3 subnet, then I need to modify public subnet with routes to FW subnets. But this module create only one RT for public subnets and does not allow to modify routes.
Here is code:
https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/master/main.tf#L126

Could you please create 3 RT for public subnet and ( if it will not add to many time ) allow to pass routes there with variables?

There is no problem to create whole AWS Network Firewall setup in my module, but I cannot use it with this module in such case.

@alekc
Copy link

alekc commented Sep 14, 2023

If a customer doesn't want to use AWS nat gw (enable_nat_gateway = false) and use own solution, they cannot because the routing table is the same (or better, they cannot avoid cross AZ traffic and related cost).

Is there advantage in keeping a single unified route table compared to one RT for every subnet?

@dimakievukraine
Copy link

@antonbabenko hi. Is there any chance to look at this issue? I cannot use your Network Firewall module with this VPC module due to single and non flexible public subnet RT. 3 public subnet route tables + adding optional firewall VPC endpoint as route would help a lot.

@bryantbiggs
Copy link
Member

The network firewall module was designed here which contains a different method for provisioning subnets. we are planning to continue to migrate that functionality over to the respective modules within this GitHub organization

@kclinden
Copy link
Contributor

Are there any branches that currently have this functionality?

@blongman-snapdocs
Copy link

Starting on my journey to add Network Firewall to my own VPCs that use this module. I'm hoping that whatever I do is easily convertible to use this module in the future.

@ligol
Copy link

ligol commented Aug 29, 2024

Hello @bryantbiggs, is there any new on this? We're currently looking to integrate the NFW into our infrastructure, but got stuck with the ompossibility to change the route table of our public subnet in order to route the traffic throught the firewall.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants