Adding policies

README.md

@@ -160,59 +160,59 @@ Available targets:
 
 | Name | Version |
 |------|---------|
-| terraform | >= 0.13.0 |
-| aws | >= 3.0 |
-| local | >= 1.3 |
+| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13.0 |
+| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 3.0 |
+| <a name="requirement_local"></a> [local](#requirement\_local) | >= 1.3 |
 
 ## Providers
 
 | Name | Version |
 |------|---------|
-| aws | >= 3.0 |
+| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 3.0 |
 
 ## Modules
 
 | Name | Source | Version |
 |------|--------|---------|
-| this | cloudposse/label/null | 0.24.1 |
+| <a name="module_this"></a> [this](#module\_this) | cloudposse/label/null | 0.24.1 |
 
 ## Resources
 
-| Name |
-|------|
-| [aws_iam_policy_document](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_policy_document) |
-| [aws_organizations_policy](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/organizations_policy) |
-| [aws_organizations_policy_attachment](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/organizations_policy_attachment) |
+| Name | Type |
+|------|------|
+| [aws_organizations_policy.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/organizations_policy) | resource |
+| [aws_organizations_policy_attachment.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/organizations_policy_attachment) | resource |
+| [aws_iam_policy_document.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_policy_document) | data source |
 
 ## Inputs
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| additional\_tag\_map | Additional tags for appending to tags\_as\_list\_of\_maps. Not added to `tags`. | `map(string)` | `{}` | no |
-| attributes | Additional attributes (e.g. `1`) | `list(string)` | `[]` | no |
-| context | Single object for setting entire context at once.<br>See description of individual variables for details.<br>Leave string and numeric variables as `null` to use default value.<br>Individual variable settings (non-null) override settings in context object,<br>except for attributes, tags, and additional\_tag\_map, which are merged. | `any` | <pre>{<br>  "additional_tag_map": {},<br>  "attributes": [],<br>  "delimiter": null,<br>  "enabled": true,<br>  "environment": null,<br>  "id_length_limit": null,<br>  "label_key_case": null,<br>  "label_order": [],<br>  "label_value_case": null,<br>  "name": null,<br>  "namespace": null,<br>  "regex_replace_chars": null,<br>  "stage": null,<br>  "tags": {}<br>}</pre> | no |
-| delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes`.<br>Defaults to `-` (hyphen). Set to `""` to use no delimiter at all. | `string` | `null` | no |
-| enabled | Set to false to prevent the module from creating any resources | `bool` | `null` | no |
-| environment | Environment, e.g. 'uw2', 'us-west-2', OR 'prod', 'staging', 'dev', 'UAT' | `string` | `null` | no |
-| id\_length\_limit | Limit `id` to this many characters (minimum 6).<br>Set to `0` for unlimited length.<br>Set to `null` for default, which is `0`.<br>Does not affect `id_full`. | `number` | `null` | no |
-| label\_key\_case | The letter case of label keys (`tag` names) (i.e. `name`, `namespace`, `environment`, `stage`, `attributes`) to use in `tags`.<br>Possible values: `lower`, `title`, `upper`.<br>Default value: `title`. | `string` | `null` | no |
-| label\_order | The naming order of the id output and Name tag.<br>Defaults to ["namespace", "environment", "stage", "name", "attributes"].<br>You can omit any of the 5 elements, but at least one must be present. | `list(string)` | `null` | no |
-| label\_value\_case | The letter case of output label values (also used in `tags` and `id`).<br>Possible values: `lower`, `title`, `upper` and `none` (no transformation).<br>Default value: `lower`. | `string` | `null` | no |
-| name | Solution name, e.g. 'app' or 'jenkins' | `string` | `null` | no |
-| namespace | Namespace, which could be your organization name or abbreviation, e.g. 'eg' or 'cp' | `string` | `null` | no |
-| regex\_replace\_chars | Regex to replace chars with empty string in `namespace`, `environment`, `stage` and `name`.<br>If not set, `"/[^a-zA-Z0-9-]/"` is used to remove all characters other than hyphens, letters and digits. | `string` | `null` | no |
-| service\_control\_policy\_description | Description of the combined Service Control Policy | `string` | `null` | no |
-| service\_control\_policy\_statements | List of Service Control Policy statements | `any` | n/a | yes |
-| stage | Stage, e.g. 'prod', 'staging', 'dev', OR 'source', 'build', 'test', 'deploy', 'release' | `string` | `null` | no |
-| tags | Additional tags (e.g. `map('BusinessUnit','XYZ')` | `map(string)` | `{}` | no |
-| target\_id | The unique identifier (ID) of the organization root, organizational unit, or account number that you want to attach the policy to | `string` | n/a | yes |
+| <a name="input_additional_tag_map"></a> [additional\_tag\_map](#input\_additional\_tag\_map) | Additional tags for appending to tags\_as\_list\_of\_maps. Not added to `tags`. | `map(string)` | `{}` | no |
+| <a name="input_attributes"></a> [attributes](#input\_attributes) | Additional attributes (e.g. `1`) | `list(string)` | `[]` | no |
+| <a name="input_context"></a> [context](#input\_context) | Single object for setting entire context at once.<br>See description of individual variables for details.<br>Leave string and numeric variables as `null` to use default value.<br>Individual variable settings (non-null) override settings in context object,<br>except for attributes, tags, and additional\_tag\_map, which are merged. | `any` | <pre>{<br>  "additional_tag_map": {},<br>  "attributes": [],<br>  "delimiter": null,<br>  "enabled": true,<br>  "environment": null,<br>  "id_length_limit": null,<br>  "label_key_case": null,<br>  "label_order": [],<br>  "label_value_case": null,<br>  "name": null,<br>  "namespace": null,<br>  "regex_replace_chars": null,<br>  "stage": null,<br>  "tags": {}<br>}</pre> | no |
+| <a name="input_delimiter"></a> [delimiter](#input\_delimiter) | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes`.<br>Defaults to `-` (hyphen). Set to `""` to use no delimiter at all. | `string` | `null` | no |
+| <a name="input_enabled"></a> [enabled](#input\_enabled) | Set to false to prevent the module from creating any resources | `bool` | `null` | no |
+| <a name="input_environment"></a> [environment](#input\_environment) | Environment, e.g. 'uw2', 'us-west-2', OR 'prod', 'staging', 'dev', 'UAT' | `string` | `null` | no |
+| <a name="input_id_length_limit"></a> [id\_length\_limit](#input\_id\_length\_limit) | Limit `id` to this many characters (minimum 6).<br>Set to `0` for unlimited length.<br>Set to `null` for default, which is `0`.<br>Does not affect `id_full`. | `number` | `null` | no |
+| <a name="input_label_key_case"></a> [label\_key\_case](#input\_label\_key\_case) | The letter case of label keys (`tag` names) (i.e. `name`, `namespace`, `environment`, `stage`, `attributes`) to use in `tags`.<br>Possible values: `lower`, `title`, `upper`.<br>Default value: `title`. | `string` | `null` | no |
+| <a name="input_label_order"></a> [label\_order](#input\_label\_order) | The naming order of the id output and Name tag.<br>Defaults to ["namespace", "environment", "stage", "name", "attributes"].<br>You can omit any of the 5 elements, but at least one must be present. | `list(string)` | `null` | no |
+| <a name="input_label_value_case"></a> [label\_value\_case](#input\_label\_value\_case) | The letter case of output label values (also used in `tags` and `id`).<br>Possible values: `lower`, `title`, `upper` and `none` (no transformation).<br>Default value: `lower`. | `string` | `null` | no |
+| <a name="input_name"></a> [name](#input\_name) | Solution name, e.g. 'app' or 'jenkins' | `string` | `null` | no |
+| <a name="input_namespace"></a> [namespace](#input\_namespace) | Namespace, which could be your organization name or abbreviation, e.g. 'eg' or 'cp' | `string` | `null` | no |
+| <a name="input_regex_replace_chars"></a> [regex\_replace\_chars](#input\_regex\_replace\_chars) | Regex to replace chars with empty string in `namespace`, `environment`, `stage` and `name`.<br>If not set, `"/[^a-zA-Z0-9-]/"` is used to remove all characters other than hyphens, letters and digits. | `string` | `null` | no |
+| <a name="input_service_control_policy_description"></a> [service\_control\_policy\_description](#input\_service\_control\_policy\_description) | Description of the combined Service Control Policy | `string` | `null` | no |
+| <a name="input_service_control_policy_statements"></a> [service\_control\_policy\_statements](#input\_service\_control\_policy\_statements) | List of Service Control Policy statements | `any` | n/a | yes |
+| <a name="input_stage"></a> [stage](#input\_stage) | Stage, e.g. 'prod', 'staging', 'dev', OR 'source', 'build', 'test', 'deploy', 'release' | `string` | `null` | no |
+| <a name="input_tags"></a> [tags](#input\_tags) | Additional tags (e.g. `map('BusinessUnit','XYZ')` | `map(string)` | `{}` | no |
+| <a name="input_target_id"></a> [target\_id](#input\_target\_id) | The unique identifier (ID) of the organization root, organizational unit, or account number that you want to attach the policy to | `string` | n/a | yes |
 
 ## Outputs
 
 | Name | Description |
 |------|-------------|
-| organizations\_policy\_arn | Amazon Resource Name (ARN) of the policy |
-| organizations\_policy\_id | The unique identifier of the policy |
+| <a name="output_organizations_policy_arn"></a> [organizations\_policy\_arn](#output\_organizations\_policy\_arn) | Amazon Resource Name (ARN) of the policy |
+| <a name="output_organizations_policy_id"></a> [organizations\_policy\_id](#output\_organizations\_policy\_id) | The unique identifier of the policy |
 <!-- markdownlint-restore -->
 
 
@@ -247,6 +247,7 @@ For additional context, refer to some of these links.
 - [Terraform Module Requirements](https://www.terraform.io/docs/registry/modules/publish.html#requirements) - HashiCorp's guidance on all the requirements for publishing a module. Meeting the requirements for publishing a module is extremely easy.
 - [Terraform `random_integer` Resource](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/integer) - The resource random_integer generates random values from a given range, described by the min and max attributes of a given resource.
 - [Terraform Version Pinning](https://www.terraform.io/docs/configuration/terraform.html#specifying-a-required-terraform-version) - The required_version setting can be used to constrain which versions of the Terraform CLI can be used with your configuration
+- [SCPs size limits](https://github.com/hashicorp/terraform-provider-aws/issues/12597) - The SCP have a size limit and creating many policies at once can result in a POLICY_CONTENT_LIMIT_EXCEEDED error
 
 
 ## Help
@@ -381,14 +382,16 @@ Check out [our other projects][github], [follow us on twitter][twitter], [apply
 ### Contributors
 
 <!-- markdownlint-disable -->
-|  [![Erik Osterman][osterman_avatar]][osterman_homepage]<br/>[Erik Osterman][osterman_homepage] | [![Andriy Knysh][aknysh_avatar]][aknysh_homepage]<br/>[Andriy Knysh][aknysh_homepage] |
-|---|---|
+|  [![Erik Osterman][osterman_avatar]][osterman_homepage]<br/>[Erik Osterman][osterman_homepage] | [![Andriy Knysh][aknysh_avatar]][aknysh_homepage]<br/>[Andriy Knysh][aknysh_homepage] | [![PePe Amengual][jamengual_avatar]][jamengual_homepage]<br/>[PePe Amengual][jamengual_homepage] |
+|---|---|---|
 <!-- markdownlint-restore -->
 
   [osterman_homepage]: https://github.com/osterman
   [osterman_avatar]: https://img.cloudposse.com/150x150/https://github.com/osterman.png
   [aknysh_homepage]: https://github.com/aknysh
   [aknysh_avatar]: https://img.cloudposse.com/150x150/https://github.com/aknysh.png
+  [jamengual_homepage]: https://github.com/jamengual
+  [jamengual_avatar]: https://img.cloudposse.com/150x150/https://github.com/jamengual.png
 
 [![README Footer][readme_footer_img]][readme_footer_link]
 [![Beacon][beacon]][website]

README.yaml

@@ -83,7 +83,9 @@ references:
   - name: "Terraform Version Pinning"
     description: "The required_version setting can be used to constrain which versions of the Terraform CLI can be used with your configuration"
     url: "https://www.terraform.io/docs/configuration/terraform.html#specifying-a-required-terraform-version"
-
+  - name: "SCPs size limits"
+    description: "The SCP have a size limit and creating many policies at once can result in a POLICY_CONTENT_LIMIT_EXCEEDED error"
+    url: "https://github.com/hashicorp/terraform-provider-aws/issues/12597"
 # Short description of this project
 description: |-
   Terraform module to provision Service Control Policies (SCP) for AWS Organizations, Organizational Units, and AWS accounts.
@@ -148,3 +150,5 @@ contributors:
     github: "osterman"
   - name: "Andriy Knysh"
     github: "aknysh"
+  - name: "PePe Amengual"
+    github: "jamengual"

catalog/account-policies.yaml

@@ -0,0 +1,7 @@
+- sid: "DenyAccountRegionDisableEnable"
+  effect: "Deny"
+  actions:
+    - "account:EnableRegion"
+    - "account:DisableRegion"
+  resources:
+    - "*"

catalog/cloudtrail-policies.yaml

@@ -0,0 +1,9 @@
+- sid: "DenyCloudTrailActions"
+  effect: "Deny"
+  actions:
+    - "cloudtrail:DeleteTrail"
+    - "cloudtrail:PutEventSelectors"
+    - "cloudtrail:StopLogging"
+    - "cloudtrail:UpdateTrail"
+  resources:
+    - "*"

catalog/cloudwatch-logs-policies.yaml

@@ -1,8 +1,20 @@
-- sid: "DenyDeletingCloudWatchLogs"
+- sid: "DenyCloudWatchDeletingLogs"
   effect: "Deny"
   actions:
     - "ec2:DeleteFlowLogs"
     - "logs:DeleteLogGroup"
     - "logs:DeleteLogStream"
   resources:
     - "*"
+
+- sid: "DenyDisablingCloudWatch"
+  effect: "Deny"
+  actions:
+    - "cloudwatch:DeleteAlarms"
+    - "cloudwatch:DeleteDashboards"
+    - "cloudwatch:DisableAlarmActions"
+    - "cloudwatch:PutDashboard"
+    - "cloudwatch:PutMetricAlarm"
+    - "cloudwatch:SetAlarmState"
+  resources:
+    - "*"

catalog/config-policies.yaml

@@ -0,0 +1,12 @@
+- sid: "DenyConfigRulesDelete"
+  effect: "Deny"
+  actions:
+    - "config:DeleteConfigRule"
+    - "config:DeleteConfigurationRecorder"
+    - "config:DeleteDeliveryChannel"
+    - "config:StopConfigurationRecorder"
+    - "config:DeleteRetentionConfiguration"
+    - "config:DeleteEvaluationResults"
+    - "config:DeleteConfigurationAggregator"
+  resources:
+    - "*"

catalog/ec2-policies.yaml

@@ -2,7 +2,7 @@
 # This policy denies instance types that aren't based on the Nitro system as documented in the following document:
 # https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/instance-types.html#ec2-nitro-instances
 #
-- sid: "DenyNonNitroInstances"
+- sid: "DenyEC2NonNitroInstances"
   effect: "Deny"
   actions:
     - "ec2:RunInstances"
@@ -83,7 +83,7 @@
 # This policy denies instance types that aren't based on the Nitro system and don't support Encryption-in-Transit as
 # described in the following document:
 # https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/data-protection.html#encryption-transit
-- sid: "DenyInstancesWithoutEncryptionInTransit"
+- sid: "DenyEC2InstancesWithoutEncryptionInTransit"
   effect: "Deny"
   actions:
     - "ec2:RunInstances"
@@ -109,3 +109,77 @@
         - r5n
   resources:
     - "arn:aws:ec2:*:*:instance/*"
+
+- sid: "DenyEC2PublicAMI"
+  effect: "Deny"
+  actions:
+    - "ec2:RunInstances"
+  condition:
+    - test: "Bool"
+      variable: "ec2:Public"
+      values:
+        - true
+  resources:
+    - "arn:aws:ec2:*::image/*"
+
+- sid: "DenyEC2AssociatePublicIp"
+  effect: "Deny"
+  actions:
+    - "ec2:RunInstances"
+  condition:
+    - test: "Bool"
+      variable: "ec2:AssociatePublicIpAddress"
+      values:
+        - true
+  resources:
+    - "arn:aws:ec2:*:*:network-interface/*"
+
+- sid: "DenyEC2AMIWithNoResourceTag"
+  effect: "Deny"
+  actions:
+    - "ec2:RunInstances"
+  condition:
+    - test: "StringNotEqualsIgnoreCase"
+      variable: "ec2:ResourceTag/${ami_tag_key}"
+      values:
+        - "${ami_tag_value}"
+  resources:
+    - "arn:aws:ec2:*::image/ami-*"
+
+- sid: "DenyEC2WithNoIMDSv2"
+  effect: "Deny"
+  actions:
+    - "ec2:RunInstances"
+  condition:
+    - test: "StringNotEquals"
+      variable: "ec2:MetadataHttpTokens"
+      values:
+        - "required"
+  resources:
+    - "arn:aws:ec2:*:*:instance/*"
+
+- sid: "DenyEC2ApiWithNoMFA"
+  effect: "Deny"
+  actions:
+    - "ec2:StopInstances"
+    - "ec2:TerminateInstances"
+    - "ec2:SendDiagnosticInterrupt"
+  condition:
+    - test: "BoolIfExists"
+      variable: "aws:MultiFactorAuthPresent"
+      values:
+        - false
+  resources:
+    - "*"
+
+- sid: "DenyEC2AMINotCreatedBy"
+  effect: "Deny"
+  actions:
+    - "ec2:RunInstances"
+  condition:
+    - test: "StringNotEquals"
+      variable: "ec2:Owner"
+      values:
+        - "${ami_creator_account}"
+  resources:
+    - "arn:aws:ec2:*::image/ami-*"

catalog/guardduty-policies.yaml

@@ -0,0 +1,43 @@
+- sid: "DenyGuardDutyDisassociation"
+  effect: "Deny"
+  actions:
+    - "guardduty:DisassociateFromMasterAccount"
+  resources:
+    - "*"
+
+- sid: "DenyDisablingGuardDuty"
+  effect: "Deny"
+  actions:
+    - "guardduty:AcceptInvitation"
+    - "guardduty:ArchiveFindings"
+    - "guardduty:CreateDetector"
+    - "guardduty:CreateFilter"
+    - "guardduty:CreateIPSet"
+    - "guardduty:CreateMembers"
+    - "guardduty:CreatePublishingDestination"
+    - "guardduty:CreateSampleFindings"
+    - "guardduty:CreateThreatIntelSet"
+    - "guardduty:DeclineInvitations"
+    - "guardduty:DeleteDetector"
+    - "guardduty:DeleteFilter"
+    - "guardduty:DeleteInvitations"
+    - "guardduty:DeleteIPSet"
+    - "guardduty:DeleteMembers"
+    - "guardduty:DeletePublishingDestination"
+    - "guardduty:DeleteThreatIntelSet"
+    - "guardduty:DisassociateFromMasterAccount"
+    - "guardduty:DisassociateMembers"
+    - "guardduty:InviteMembers"
+    - "guardduty:StartMonitoringMembers"
+    - "guardduty:StopMonitoringMembers"
+    - "guardduty:TagResource"
+    - "guardduty:UnarchiveFindings"
+    - "guardduty:UntagResource"
+    - "guardduty:UpdateDetector"
+    - "guardduty:UpdateFilter"
+    - "guardduty:UpdateFindingsFeedback"
+    - "guardduty:UpdateIPSet"
+    - "guardduty:UpdatePublishingDestination"
+    - "guardduty:UpdateThreatIntelSet"
+  resources:
+    - "*"

catalog/iam-policies.yaml

@@ -1,13 +1,12 @@
-- sid: "DenyCreatingIAMUsers"
+- sid: "DenyIAMCreatingUsers"
   effect: "Deny"
   actions:
     - "iam:CreateUser"
     - "iam:CreateAccessKey"
   resources:
     - "*"
 
-
-- sid: "ProtectIAMRoles"
+- sid: "DenyIAMRolesChanges"
   effect: "Deny"
   actions:
     - "iam:AttachRolePolicy"
@@ -22,3 +21,33 @@
     - "iam:UpdateRoleDescription"
   resources:
     - "*"
+
+- sid: "DenyIAMNoMFA"
+  effect: "Deny"
+  not_actions:
+    - "iam:CreateVirtualMFADevice"
+    - "iam:EnableMFADevice"
+    - "iam:GetUser"
+    - "iam:ListMFADevices"
+    - "iam:ListVirtualMFADevices"
+    - "iam:ResyncMFADevice"
+    - "sts:GetSessionToken"
+  condition:
+    - test: "BoolIfExists"
+      variable: "aws:MultiFactorAuthPresent"
+      values:
+        - false
+  resources:
+    - "*"
+
+- sid: "DenyIAMRootAccount"
+  effect: "Deny"
+  actions:
+    - "*"
+  condition:
+    - test: "StringLike"
+      variable: "aws:PrincipalArn"
+      values:
+        - "arn:aws:iam::*:root"
+  resources:
+    - "*"

catalog/lambda-policies.yaml

@@ -0,0 +1,12 @@
+- sid: "DenyLambdaWithoutVpc"
+  effect: "Deny"
+  actions:
+    - "lambda:CreateFunction"
+    - "lambda:UpdateFunctionConfiguration"
+  condition:
+    - test: "Null"
+      variable: "lambda:VpcIds"
+      values:
+        - true
+  resources:
+    - "*"