From 3072178016d5d523c0b39034de0a3f894ce165f0 Mon Sep 17 00:00:00 2001
From: Matthew Ridehalgh <matthew@edispark.com>
Date: Mon, 14 Dec 2020 22:25:40 +0000
Subject: [PATCH] feat: upgrade terraform aws provider (#27)

---
 .github/workflows/master.yml | 3 ++-
 .github/workflows/pr.yml     | 3 ++-
 .pre-commit-config.yaml      | 4 ++--
 README.md                    | 4 ++--
 iam.tf                       | 2 +-
 main.tf                      | 2 +-
 versions.tf                  | 2 +-
 7 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/master.yml b/.github/workflows/master.yml
index 90a1843..02b69e8 100644
--- a/.github/workflows/master.yml
+++ b/.github/workflows/master.yml
@@ -26,7 +26,8 @@ jobs:
         run: |
           pip install pre-commit
           curl -L "$(curl -s https://api.github.com/repos/terraform-linters/tflint/releases/latest | grep -o -E "https://.+?_linux_amd64.zip")" > tflint.zip && unzip tflint.zip && rm tflint.zip && sudo mv tflint /usr/bin/
-          env GO111MODULE=on go get -u github.com/liamg/tfsec/cmd/tfsec && go get github.com/segmentio/terraform-docs@v0.9.1
+          curl -L "$(curl -s https://api.github.com/repos/tfsec/tfsec/releases/latest | grep -o -E "https://.+?tfsec-linux-amd64")" > tfsec && chmod +x tfsec && sudo mv tfsec /usr/bin/
+          curl -L "$(curl -s https://api.github.com/repos/terraform-docs/terraform-docs/releases/tags/v0.9.1 | grep -o -E "https://.+?-linux-amd64")" > terraform-docs && chmod +x terraform-docs && sudo mv terraform-docs /usr/bin/
           pre-commit install
           pre-commit install-hooks
       - name: Run pre-commit tests
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
index 56a3ecb..c3d8e51 100644
--- a/.github/workflows/pr.yml
+++ b/.github/workflows/pr.yml
@@ -21,7 +21,8 @@ jobs:
         run: |
           pip install pre-commit
           curl -L "$(curl -s https://api.github.com/repos/terraform-linters/tflint/releases/latest | grep -o -E "https://.+?_linux_amd64.zip")" > tflint.zip && unzip tflint.zip && rm tflint.zip && sudo mv tflint /usr/bin/
-          env GO111MODULE=on go get -u github.com/liamg/tfsec/cmd/tfsec && go get github.com/segmentio/terraform-docs@v0.9.1
+          curl -L "$(curl -s https://api.github.com/repos/tfsec/tfsec/releases/latest | grep -o -E "https://.+?tfsec-linux-amd64")" > tfsec && chmod +x tfsec && sudo mv tfsec /usr/bin/
+          curl -L "$(curl -s https://api.github.com/repos/terraform-docs/terraform-docs/releases/tags/v0.9.1 | grep -o -E "https://.+?-linux-amd64")" > terraform-docs && chmod +x terraform-docs && sudo mv terraform-docs /usr/bin/
           pre-commit install
           pre-commit install-hooks
       - name: Run pre-commit tests
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 1581078..adeeae8 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -1,6 +1,6 @@
 repos:
   - repo: git://github.com/pre-commit/pre-commit-hooks
-    rev: v2.1.0
+    rev: v3.3.0
     hooks:
       - id: check-merge-conflict
       - id: check-json
@@ -13,7 +13,7 @@ repos:
         args:
           - --allow-missing-credentials
   - repo: git://github.com/antonbabenko/pre-commit-terraform
-    rev: v1.31.0
+    rev: v1.45.0
     hooks:
       - id: terraform_fmt
       - id: terraform_docs
diff --git a/README.md b/README.md
index 44dd5e5..3a5c952 100644
--- a/README.md
+++ b/README.md
@@ -38,14 +38,14 @@ module "example_module_test" {
 | Name | Version |
 |------|---------|
 | terraform | >= 0.12.26 |
-| aws | >= 2.0 |
+| aws | >= 3.0 |
 | local | 1.4.0 |
 
 ## Providers
 
 | Name | Version |
 |------|---------|
-| aws | >= 2.0 |
+| aws | >= 3.0 |
 | local | 1.4.0 |
 
 ## Inputs
diff --git a/iam.tf b/iam.tf
index 1d9478d..5a921e1 100644
--- a/iam.tf
+++ b/iam.tf
@@ -37,7 +37,7 @@ data "aws_iam_policy_document" "lambda" {
     ]
 
     resources = [
-      aws_cloudwatch_log_group.lambda_log_group.arn
+      format("%s:*", aws_cloudwatch_log_group.lambda_log_group.arn)
     ]
   }
 
diff --git a/main.tf b/main.tf
index 512f21d..b5cc025 100644
--- a/main.tf
+++ b/main.tf
@@ -28,7 +28,7 @@ resource "aws_lambda_function" "draining_lambda" {
   }
 
   filename         = data.local_file.lambda_zip.filename
-  source_code_hash = data.local_file.lambda_zip.content_base64
+  source_code_hash = filebase64sha256(data.local_file.lambda_zip.filename)
 
   tags = var.tags
 }
diff --git a/versions.tf b/versions.tf
index 181a45f..918a305 100644
--- a/versions.tf
+++ b/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 2.0"
+      version = ">= 3.0"
     }
     local = {
       source  = "hashicorp/local"