Merge pull request #4 from theonestack/inline-config

inline the cloudfront config
theonestack · Nov 19, 2020 · 50b804a · 50b804a
2 parents 163da07 + 1a10f41
commit 50b804a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/s3-cloudfront.cfhighlander.rb b/s3-cloudfront.cfhighlander.rb
@@ -8,7 +8,7 @@
     ComponentParam 'DnsDomain', isGlobal: true
   end
 
-  Component name: cloudfront_component_name, template: 'cloudfront@master.snapshot', render: Inline do
+  Component name: cloudfront_component_name, template: 'cloudfront@master.snapshot', render: Inline, config: @config do
     parameter name: "s3bucketOriginDomainName", value: FnGetAtt('Bucket', 'DomainName')
   end
 

diff --git a/s3-cloudfront.cfndsl.rb b/s3-cloudfront.cfndsl.rb
@@ -8,7 +8,7 @@
         Effect: 'Allow',
         Action: 's3:GetObject',
         Resource: FnJoin('', [ 'arn:aws:s3:::', Ref("Bucket"), '/*']),
-        Principal: { CanonicalUser: FnGetAtt('s3bucketOriginAccessIdentity', 'S3CanonicalUserId') }
+        Principal: { CanonicalUser: { "Fn::GetAtt" => ['s3bucketOriginAccessIdentity', 'S3CanonicalUserId'] }}
       }
     ]
   }