Merge pull request #15 from amavrogiannis/feat/reformat-terraform

Feat/reformat terraform p2
amavrogiannis · Mar 29, 2024 · ac60f2d · ac60f2d
2 parents 1e8141c + 6a3c800
commit ac60f2d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/terraform_prod.yaml b/.github/workflows/terraform_prod.yaml
@@ -3,14 +3,13 @@ name: "Production - Terraform Infrastructure"
 on:
  push:
    branches:
-    - feature/**
     - main
    paths:
     - infra/envs/prod/**
     - infra/modules/**
  pull_request:
    branches:
-    - main
+    - prod
    paths:
     - infra/envs/prod/**
     - infra/modules/**

diff --git a/infra/modules/web/cloudfront.tf b/infra/modules/web/cloudfront.tf
@@ -97,8 +97,8 @@ resource "aws_cloudfront_distribution" "this" {
 
   #First, create certificate, before pasting the ARN here. 
   viewer_certificate {
-    acm_certificate_arn            = data.aws_acm_certificate.this.arn
-    cloudfront_default_certificate = false
+    # acm_certificate_arn            = data.aws_acm_certificate.this.arn
+    cloudfront_default_certificate = true
     minimum_protocol_version       = "TLSv1.2_2021"
     ssl_support_method             = "sni-only"
   }