Skip to content

Commit

Permalink
Use Origin Access Control (#65)
Browse files Browse the repository at this point in the history
Use Origin Access Control

* Swap from Origin Access Identity to Origin Access Control
* Bump version

Issue #63
  • Loading branch information
ConnorKirk committed Aug 10, 2023
1 parent 2f5fe54 commit 40e0877
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 11 deletions.
25 changes: 15 additions & 10 deletions templates/cloudfront-site.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,14 @@ Resources:
PolicyDocument:
Version: '2012-10-17'
Statement:
- Action:
- s3:GetObject
- Action: s3:GetObject
Principal:
Service: 'cloudfront.amazonaws.com'
Effect: Allow
Resource: !Sub '${S3BucketRootArn}/*'
Principal:
CanonicalUser: !GetAtt CloudFrontOriginAccessIdentity.S3CanonicalUserId
Condition:
StringEquals:
'AWS:SourceArn': !Sub 'arn:aws:cloudfront::${AWS::AccountId}:distribution/${CloudFrontDistribution}'

CloudFrontDistribution:
Type: AWS::CloudFront::Distribution
Expand Down Expand Up @@ -86,8 +88,8 @@ Resources:
Origins:
- DomainName: !Ref 'S3BucketRootName'
Id: !Sub 'S3-${AWS::StackName}-root'
S3OriginConfig:
OriginAccessIdentity: !Sub 'origin-access-identity/cloudfront/${CloudFrontOriginAccessIdentity}'
OriginAccessControlId: !Ref OriginAccessControl
S3OriginConfig: {}
PriceClass: 'PriceClass_All'
ViewerCertificate:
AcmCertificateArn: !Ref 'CertificateArn'
Expand All @@ -97,11 +99,14 @@ Resources:
- Key: Solution
Value: ACFS3

CloudFrontOriginAccessIdentity:
Type: AWS::CloudFront::CloudFrontOriginAccessIdentity
OriginAccessControl:
Type: AWS::CloudFront::OriginAccessControl
Properties:
CloudFrontOriginAccessIdentityConfig:
Comment: !Sub 'CloudFront OAI for ${SubDomain}.${DomainName}'
OriginAccessControlConfig:
Name: !Sub 'oac-${AWS::StackName}-${AWS::Region}'
OriginAccessControlOriginType: s3
SigningBehavior: always
SigningProtocol: sigv4

Route53RecordSetGroup:
Type: AWS::Route53::RecordSetGroup
Expand Down
2 changes: 1 addition & 1 deletion templates/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Metadata:
Mappings:
Solution:
Constants:
Version: 'v0.9'
Version: 'v0.10'

Rules:
OnlyUsEast1:
Expand Down

0 comments on commit 40e0877

Please sign in to comment.