Skip to content

Commit ccf418e

Browse files
committed
fix: ch16 was broken because of new default S3 security settings
1 parent 73576c3 commit ccf418e

File tree

2 files changed

+16
-0
lines changed

2 files changed

+16
-0
lines changed

chapter16/template.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,14 @@ Resources:
122122
Type: 'AWS::S3::Bucket'
123123
Properties:
124124
BucketName: !Sub 'imagery-${AWS::AccountId}'
125+
OwnershipControls:
126+
Rules:
127+
- ObjectOwnership: BucketOwnerPreferred
128+
PublicAccessBlockConfiguration:
129+
BlockPublicAcls: false
130+
BlockPublicPolicy: true
131+
IgnorePublicAcls: false
132+
RestrictPublicBuckets: true
125133
WebsiteConfiguration:
126134
ErrorDocument: error.html
127135
IndexDocument: index.html

chapter17/url2png.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,14 @@ Resources:
169169
Type: 'AWS::S3::Bucket'
170170
Properties:
171171
BucketName: !Sub 'ch17-url2png-${AWS::AccountId}'
172+
OwnershipControls:
173+
Rules:
174+
- ObjectOwnership: BucketOwnerPreferred
175+
PublicAccessBlockConfiguration:
176+
BlockPublicAcls: false
177+
BlockPublicPolicy: true
178+
IgnorePublicAcls: false
179+
RestrictPublicBuckets: true
172180
SQSQueue:
173181
Type: 'AWS::SQS::Queue'
174182
Properties:

0 commit comments

Comments
 (0)