Skip to content

Add event rule pattern detail and control finding generator variable - #59

Open
Alex Boyd (aaboyd) wants to merge 4 commits into
cloudposse:mainfrom
minwareco:add-event-rule-pattern-detail
Open

Add event rule pattern detail and control finding generator variable#59
Alex Boyd (aaboyd) wants to merge 4 commits into
cloudposse:mainfrom
minwareco:add-event-rule-pattern-detail

Conversation

@aaboyd

Copy link
Copy Markdown

what

Add two variables for better customization of the module.

why

cloudwatch_event_rule_pattern_detail - allow a more specific detail pattern in order to filter events sent through the SNS topic. Currently using to only match events which have failed and are 'HIGH' and 'CRITICAL'.

control_finding_generator - optionality that exists on aws_securityhub_account. Ensures that users of this module don't sacrifice functionality.

@zestrells

Zachary Estrella (zestrells) commented Nov 21, 2024

Copy link
Copy Markdown

Any chance we can get this prioritized / merged? Looking for this feature now, as enabling the default SNS with the module sends too many alerts

Comment thread variables.tf
check applies to multiple enabled standards.
DOC
type = string
default = "SECURITY_CONTROL"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to add a validation block here to check if the values inputted are correct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage Needs triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants