Skip to content

Commit 471671f

Browse files
authored
data-setup.sh: stop touching S3 block-public-access (#90)
Drop ensure_s3_block_public_access entirely. The script no longer opines on bucket BPA -- existing buckets keep whatever they have, new buckets get the AWS account-level default.
1 parent 7355e7f commit 471671f

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

scripts/data-setup.sh

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -180,14 +180,6 @@ ensure_s3_bucket() {
180180
--tagging "$(tags_s3_envelope ingest-bucket)" >/dev/null
181181
}
182182

183-
ensure_s3_block_public_access() {
184-
log "applying block-public-access to $BUCKET"
185-
aws_cli s3api put-public-access-block \
186-
--bucket "$BUCKET" \
187-
--public-access-block-configuration \
188-
BlockPublicAcls=true,IgnorePublicAcls=true,BlockPublicPolicy=true,RestrictPublicBuckets=true
189-
}
190-
191183
ensure_s3_lifecycle() {
192184
log "applying lifecycle (expire after $BUCKET_LIFECYCLE_DAYS days) to $BUCKET"
193185
cfg=$(jq -nc --argjson d "$BUCKET_LIFECYCLE_DAYS" '{
@@ -390,7 +382,6 @@ main() {
390382
# ---- storage ----------------------------------------------------------
391383
queue_url=$(ensure_sqs_queue)
392384
ensure_s3_bucket
393-
ensure_s3_block_public_access
394385
ensure_s3_lifecycle
395386
ensure_sqs_policy "$queue_url"
396387
ensure_s3_notification

0 commit comments

Comments
 (0)