This repository was archived by the owner on Apr 29, 2024. It is now read-only.

Commit c7c871d

authored

Fix restricting access assets.

As it stands, the `S3OriginIdentityExample` has not been associated with the `Cloudfrontdistribution` resulting in access denied errors when attempting to fetch assets. This is fixed creating said association. Verified through rdok/serverless-laravel@acaddcc

1 parent f4277b6 commit c7c871dCopy full SHA for c7c871d

File tree

1 file changed

-2

lines changed

0.4-Building-A-Serverless-Laravel-App-With-AWS-SAM
- template.yaml

1 file changed

-2

lines changed

`‎0.4-Building-A-Serverless-Laravel-App-With-AWS-SAM/template.yaml`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,8 @@ Resources:`
`99`	`99`	`# The assets (S3)`
`100`	`100`	`- Id: Assets`
`101`	`101`	`DomainName: !GetAtt Assets.RegionalDomainName`
`102`		`- S3OriginConfig: {} # this key is required to tell CloudFront that this is an S3 origin, even though nothing is configured`
	`102`	`+ S3OriginConfig:`
	`103`	`+ OriginAccessIdentity: !Sub "origin-access-identity/cloudfront/${S3OriginIdentityExample}"`
`103`	`104`	`# The default behavior is to send everything to AWS Lambda`
`104`	`105`	`DefaultCacheBehavior:`
`105`	`106`	`AllowedMethods: [GET, HEAD, OPTIONS, PUT, POST, PATCH, DELETE]`
`@@ -140,4 +141,4 @@ Resources:`
`140`	`141`	`Outputs:`
`141`	`142`	`Domain:`
`142`	`143`	`Description: 'CloudFront domain name'`
`143`		`- Value: !GetAtt Cloudfrontdistribution.DomainName`
	`144`	`+ Value: !GetAtt Cloudfrontdistribution.DomainName`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit c7c871d

File tree

1 file changed

1 file changed

`‎0.4-Building-A-Serverless-Laravel-App-With-AWS-SAM/template.yaml`

0 commit comments