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

Change filter expressions as per CIS Benchmark guide #9

Closed
rsgithub123 opened this issue Jul 19, 2019 · 2 comments
Closed

Change filter expressions as per CIS Benchmark guide #9

rsgithub123 opened this issue Jul 19, 2019 · 2 comments

Comments

@rsgithub123
Copy link
Contributor

We are using a tool that audits the alarms implemented in our environment as per CIS Banchmark guide ( https://d0.awsstatic.com/whitepapers/compliance/AWS_CIS_Foundations_Benchmark.pdf).
It matches exact expression that are defined in CIS Banchmark including parentheses. There are 3 filters in alarms.tf that have mismatch.
Please change the following filter expressions as per CIS Banchmark guide.

3.2 Ensure a log metric filter and alarm exist for Management Console sign-in without MFA (Scored)

Page 92:

CIS Banchmark Guide: "{ ($.eventName = "ConsoleLogin") &&($.additionalEventData.MFAUsed != "Yes") }"
alarms.tf : "{ $.eventName = "ConsoleLogin" && $.additionalEventData.MFAUsed = "No" }"

3.7 Ensure a log metric filter and alarm exist for disabling or scheduled deletion of customer created CMKs (Scored)

Page 112:

CIS Banchmark Guide: "{($.eventSource = kms.amazonaws.com) && (($.eventName=DisableKey)||($.eventName=ScheduleKeyDeletion)) }"
alarms.tf : "{ $.eventSource = kms* && $.errorMessage = "* is pending deletion."}"

3.9 Ensure a log metric filter and alarm exist for AWS Config configuration changes (Scored)

page 120:

CIS Banchmark Guide: "{ ($.eventSource = config.amazonaws.com) &&(($.eventName=StopConfigurationRecorder)||($.eventName=DeleteDeliveryChannel)||($.eventName=PutDeliveryChannel)||($.eventName=PutConfigurationRecorder))}"
alarms.tf : "{ $.eventSource = config.amazonaws.com && (($.eventName=StopConfigurationRecorder)||($.eventName=DeleteDeliveryChannel) ||($.eventName=PutDeliveryChannel)||($.eventName=PutConfigurationRecorder)) }"

@mardinyadegar
Copy link
Contributor

This issue should be closed as it has been fixed now.

@Gowiem
Copy link
Member

Gowiem commented Oct 28, 2020

Closing out as this seems to have been fixed for a long while. Thanks folks.

@Gowiem Gowiem closed this as completed Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants