Skip to content

Commit

Permalink
Merge branch 'fix-sns-bug' of github.com:schubergphilis/terraform-aws…
Browse files Browse the repository at this point in the history
…-mcaf-landing-zone into fix-sns-bug
  • Loading branch information
marwinbaumannsbp committed Jan 5, 2021
2 parents 9f8f8aa + 8cc602d commit 3989199
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ module "landing_zone" {
| aws\_require\_imdsv2 | Enable SCP which requires EC2 instances to use V2 of the Instance Metadata Service | `bool` | `true` | no |
| datadog | Datadog integration options for the core accounts | <pre>object({<br> api_key = string<br> enable_integration = bool<br> install_log_forwarder = bool<br> site_url = string<br> })</pre> | `null` | no |
| monitor\_iam\_access | List of IAM Identities that should have their access monitored | <pre>list(object({<br> account = string<br> name = string<br> type = string<br> }))</pre> | `null` | no |
| sns\_security\_subscription | Aggregated security SNS topic subscription options | <pre>list(object({<br> endpoint = string<br> protocol = string<br> }))</pre> | `null` | no |
| sns\_aws\_config\_subscription | Subscription options for the aws-controltower-AggregateSecurityNotifications (AWS Config) SNS topic | <pre>map(object({<br> endpoint = string<br> protocol = string<br> }))</pre> | `{}` | no |

## Outputs

Expand Down
1 change: 0 additions & 1 deletion modules/security_hub/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ Terraform module to setup and manage AWS Security Hub.
| member\_accounts | A map of accounts that should be added as SecurityHub Member Accounts (format: account\_id = email) | `map(string)` | `{}` | no |
| product\_arns | A list of the ARNs of the products you want to import into Security Hub | `list(string)` | `[]` | no |
| region | The name of the AWS region where SecurityHub will be enabled | `string` | `"eu-west-1"` | no |
| sns\_subscription | Aggregated security SNS topic subscription options | <pre>list(object({<br> account_id = string<br> endpoint = string<br> protocol = string<br> }))</pre> | `null` | no |

## Outputs

Expand Down

0 comments on commit 3989199

Please sign in to comment.