search

fivexl / terraform-aws-cloudtrail-to-slack

Parse AWS CloudTrail events and send alerts to Slack for events that match pre-configured rules
https://registry.terraform.io/modules/fivexl/cloudtrail-to-slack/aws/latest
Apache License 2.0
93 stars 26 forks source link

Add Optional filter_prefix Variable #24

Closed jcua-helpscout closed 2 years ago

jcua-helpscout commented 2 years ago

Our s3 URI looks like this:

s3://test/CloudTrail/AWSLogs/.../

But in the aws_s3_bucket_notification resource "bucket_notification", the filter_prefix is hardcoded to "AWSLogs/" which does not work for us. The PR just aims to make the filter_prefix an optional variable and still make "AWSLogs/" as the default value.

jcua-helpscout commented 2 years ago

Need to add more tweaks to customize what we want.