Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

downgrade cloudfront to 2.9.3 #42

Closed
wants to merge 2 commits into from
Closed

downgrade cloudfront to 2.9.3 #42

wants to merge 2 commits into from

Conversation

7hunderbird
Copy link
Collaborator

Let's try to rewind the clock on CloudFront

Copy link

mapped Expected Changes

updated cloudfront-distribution › E15V1JM5GZXBKB
--- current
+++ planned
@@ -95,22 +95,21 @@
       custom_header: []
       custom_origin_config: []
       domain_name: s3-one-apt-catfish.s3.eu-west-2.amazonaws.com
-      origin_access_control_id: ""
-      origin_id: s3_one
+      origin_access_control_id: E23BFWHZQZKBPZ
+      origin_id: s3_oac
       origin_path: ""
       origin_shield: []
-      s3_origin_config:
-        - origin_access_identity: origin-access-identity/cloudfront/E2X23HICB2NVF3
+      s3_origin_config: []
     - connection_attempts: 3
       connection_timeout: 10
       custom_header: []
       custom_origin_config: []
       domain_name: s3-one-apt-catfish.s3.eu-west-2.amazonaws.com
-      origin_access_control_id: E23BFWHZQZKBPZ
-      origin_id: s3_oac
+      origin_id: s3_one
       origin_path: ""
       origin_shield: []
-      s3_origin_config: []
+      s3_origin_config:
+        - origin_access_identity: origin-access-identity/cloudfront/E2X23HICB2NVF3
     - connection_attempts: 3
       connection_timeout: 10
       custom_header:
@@ -126,7 +116,6 @@
           origin_read_timeout: 30
           origin_ssl_protocols:
             - TLSv1
-            - TLSv1.1
             - TLSv1.2
       domain_name: appsync.terraform-aws-modules.modules.tf
       origin_access_control_id: ""

Blast Radius

items Items edges Edges
153 146

Open in Overmind

@7hunderbird
Copy link
Collaborator Author

I have truly been lucky to have this PR number. 🙇🏻 🥂

I want to try a new PR where I only make the change to the TLS and see what the difference is. Closing this one for now.

@7hunderbird 7hunderbird mentioned this pull request Dec 20, 2023
@dylanratcliffe dylanratcliffe deleted the tbird-changes branch April 18, 2024 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant