-
Notifications
You must be signed in to change notification settings - Fork 63
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
Add/Update AWS Audit Manager > Control Tower controls. Closes #694 #705
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Priyanka-Chatterjee-2000 please have a look at the comments. Please also take a look at the additional feedback as follows:
Formatting is incorrect, please update it for the following files:
- disallow_instance.sp
- disallow_internet_connection.sp
- disallow_public_access.sp
- ebs_checks.sp
- multi_factor_authentication.sp
Please end the description for the following with a .
- audit_manager_control_tower_disallow_internet_connection_2_0_1
- audit_manager_control_tower_ebs_checks_1_0_2
Co-authored-by: Madhushree Ray <65847563+madhushreeray30@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Unique controls added:
Unique controls removed: