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

feat(aws): CloudTrail global service events rule #1401

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

aisha-als
Copy link
Contributor

Adding a new rule to check if Global Service Events are enabled in CloudTrail and have not been turned off as this value is enabled by default.

Links:
https://docs.aws.amazon.com/awscloudtrail/latest/userguide/cloudtrail-concepts.html#cloudtrail-concepts-global-service-events
https://registry.terraform.io/providers/rgeraskin/aws2/latest/docs/resources/cloudtrail#include_global_service_events

Also some minor fixes to example docs.

@aisha-als aisha-als force-pushed the cloudtrail-global-services branch 6 times, most recently from a4f406f to 0c828fd Compare July 23, 2023 00:11
BadExample:
GoodExample:
Copy link
Member

Choose a reason for hiding this comment

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

I'm curious why this has changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe this should say GoodExample, after updating it on the policy it still needs to be updated on the auto generated docs. Updated from this policy as part of this PR: https://github.com/aquasecurity/defsec/blob/master/rules/cloud/policies/aws/cloudtrail/enable_all_regions.cf.go

BadExample:
GoodExample:
Copy link
Member

Choose a reason for hiding this comment

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

ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Similar to my previous comment, the policy script has been updated as part of this PR and this should say GoodExample:
https://github.com/aquasecurity/defsec/blob/master/rules/cloud/policies/aws/cloudtrail/enable_at_rest_encryption.cf.go

BadExample:
GoodExample:
Copy link
Member

Choose a reason for hiding this comment

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

ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Similar to my previous comment, the policy script has been updated as part of this PR and this should say GoodExample:
https://github.com/aquasecurity/defsec/blob/master/rules/cloud/policies/aws/cloudtrail/enable_log_validation.cf.go

@github-actions
Copy link

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the stale Stale issues will be closed within 7 days of this label being assigned label Sep 14, 2023
@aisha-als
Copy link
Contributor Author

Thanks for reviewing and apologies for the late reply as I have been away.

@aisha-als aisha-als force-pushed the cloudtrail-global-services branch 2 times, most recently from eabb8e3 to 1e9e05d Compare September 18, 2023 16:14
@github-actions github-actions bot removed the stale Stale issues will be closed within 7 days of this label being assigned label Sep 19, 2023
Copy link

github-actions bot commented Nov 9, 2023

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the stale Stale issues will be closed within 7 days of this label being assigned label Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Stale issues will be closed within 7 days of this label being assigned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants