terraform-aws-guardduty-notifications

Module now requires using Terraform version 1.0 or higher.

Because the notify-slack Terraform module outputs only the name and ARN for the slack topic it creates, having this module take an SNS topic object makes consuming these outputs difficult (you have to use a data source or create the SNS topic separately). As a result, we've changed the inputs to be ARNs (the PagerDuty input was also changed to be consistent). We also changed the name of the inputs accordingly.

See the README for usage and upgrade notes.

This release focuses on:

• Updating CircleCI image
• Precommit plugins
• Upgrade TF providers
• Allow for TF 0.14
• Cleanup circleci yaml

pins merge from #39 trussworks/am-update-circle-ci

use if need terraform 12

The v3.0.0 release had a bug where there were no defaults for the Slack and PagerDuty SNS topics, so you still had to specify them even if you didn't use them. This has been remedied.

This release makes the GuardDuty detector, Slack notifications, and PagerDuty notifications all optional. It also changes the names of the parameters for providing the SNS topics for notifications to better reflect what they actual expect as an argument. These are breaking changes; see the README for instructions on how to update from v2 to v3.

Simply updating the README to reflect a TF 0.12 version of this module.

Due to a way Terraform 0.12 handles data sources, data lookups were replaced with object variables. Also added a basic Terratest to confirm Terraform exists cleanly.

Minor syntax changes to make the code compatible with both Terraform 0.11 and 0.12.