aws-samples / aws-ct-vpc-flowlogs

MIT No Attribution
10 stars 8 forks source link

Issue : CfCT - duplicate flowlogs on Subnet and VPC during first run #8

Closed wellsiau-aws closed 2 years ago

wellsiau-aws commented 2 years ago

For CfCT mode, we trigger first-run to inspect all VPCs and Subnets.

If the DefaultTrafficLoggingMode is set to enable flow logs (all, reject, accept), then there will be duplicate on both VPC and Subnet level.

Proposed fix:

DefaultTrafficLoggingMode should only apply to VPC resource. During tag parsing, check the resource type and set default value accordingly.

wellsiau-aws commented 2 years ago

Ref to PR for CfCT (https://github.com/aws-samples/aws-ct-vpc-flowlogs/pull/5)

str3tch commented 2 years ago

this is fixed with the merged PR now is it not @wellsiau-aws ?

wellsiau-aws commented 2 years ago

@str3tch , yes you are correct, I will close this issue now. Thanks!