search

obsrvbl-oss / flowlogs-reader

Command line tool and Python library for working with AWS VPC Flow Logs
Apache License 2.0
138 stars 23 forks source link

Skip TGW records #58

Closed tejasnadgoud closed 1 year ago

tejasnadgoud commented 1 year ago

Ref: https://github.com/obsrvbl/devops/issues/11665

Problem-statement: The above issue states that the code handles vpc flow data with default format but does not take in consideration if transit-gateway is enabled. So when input data contains mixed records including transitgateway records, it errors out on fields contains tgw-logs. Since transit-gateway logs are not supported in flowlogs-reader, maybe we can skip those fields instead of parsing them.

This PR intends to safely ignore all fields that contain Transit Gateway records.

bbayles commented 1 year ago

What if support for the incremental fields was added instead?

BThacker commented 1 year ago

Closing as going a different direction