fix: CloudFront behavior for ViewerProtocolPolicy (aws#615)

This was a typo in the original CloudFront L2 - this just fixes it and updates the tests for it.
moofish32 · Aug 22, 2018 · 24ce10c · 24ce10c
1 parent d0a3c7e
commit 24ce10c
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/packages/@aws-cdk/aws-cloudfront/lib/web_distribution.ts b/packages/@aws-cdk/aws-cloudfront/lib/web_distribution.ts
@@ -22,7 +22,7 @@ export enum PriceClass {
  */
 export enum ViewerProtocolPolicy {
     HTTPSOnly = "https-only",
-    RedirectToHTTPS = "https-only",
+    RedirectToHTTPS = "redirect-to-https",
     AllowAll = "allow-all"
 }
 

diff --git a/packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront-custom.expected.json b/packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront-custom.expected.json
@@ -21,7 +21,7 @@
               "QueryString": false
             },
             "TargetOriginId": "origin1",
-            "ViewerProtocolPolicy": "https-only"
+            "ViewerProtocolPolicy": "redirect-to-https"
           },
           "DefaultRootObject": "index.html",
           "Enabled": true,

diff --git a/packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront.expected.json b/packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront.expected.json
@@ -24,7 +24,7 @@
               "QueryString": false
             },
             "TargetOriginId": "origin1",
-            "ViewerProtocolPolicy": "https-only"
+            "ViewerProtocolPolicy": "redirect-to-https"
           },
           "DefaultRootObject": "index.html",
           "Enabled": true,

diff --git a/packages/@aws-cdk/aws-cloudfront/test/test.basic.ts b/packages/@aws-cdk/aws-cloudfront/test/test.basic.ts
@@ -53,7 +53,7 @@ export = {
                     "QueryString": false
                   },
                   "TargetOriginId": "origin1",
-                  "ViewerProtocolPolicy": "https-only"
+                  "ViewerProtocolPolicy": "redirect-to-https"
                 },
                 "DefaultRootObject": "index.html",
                 "Enabled": true,
@@ -150,7 +150,7 @@ export = {
                         "HEAD"
                       ],
                       "TargetOriginId": "origin1",
-                      "ViewerProtocolPolicy": "https-only",
+                      "ViewerProtocolPolicy": "redirect-to-https",
                       "ForwardedValues": {
                         "QueryString": false,
                         "Cookies" : { "Forward" : "none"}
@@ -224,7 +224,7 @@ export = {
                         "HEAD"
                       ],
                       "TargetOriginId": "origin1",
-                      "ViewerProtocolPolicy": "https-only",
+                      "ViewerProtocolPolicy": "redirect-to-https",
                       "ForwardedValues": {
                         "QueryString": false,
                         "Cookies" : { "Forward" : "none"}