Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Commit e532eda

Browse files
authored
Merge pull request #379 from aws-solutions/dev-sandim
Update S3 Object Ownership
2 parents ebd9691 + 1d5b904 commit e532eda

File tree

5 files changed

+9
-4
lines changed

5 files changed

+9
-4
lines changed

CHANGELOG.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ All notable changes to this project will be documented in this file.
55
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
77

8-
## [Unreleased]
8+
## [2.1.0] - 2023-04-19
99

1010
### Added
1111

@@ -18,11 +18,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1818
* The "Download Data" button on the **Celebrities** tab works now.
1919
* Fixed upload of WebVTT files as "Existing Subtitles".
2020
* Incorrect Cloudformation template in public bucket (#365)
21+
* Updated object ownership configuration on ContentAnalysisWebsiteBucket
2122

2223
### Changed
2324

2425
* Removed Hawkeye scan from github actions (deprecated)
25-
* Upgrade Media Insights on AWS dependency to v5.1.0
26+
* Upgrade Media Insights on AWS dependency to v5.1.1
2627
* Refactored some code for maintainability
2728
* Updated e2e tests for bug fixes and compatibility with new Media Insights on AWS version
2829
* Updated references to "Media Insights Engine" to "Media Insights on AWS"

deployment/build-s3-dist.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,6 @@ msg "- Profile: ${profile}"
129129

130130
echo ""
131131
sleep 3
132-
s3domain="s3.$region.amazonaws.com"
133132

134133
# Check if region is supported:
135134
if [ "$region" != "us-east-1" ] &&

deployment/content-localization-on-aws-web.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,9 @@ Resources:
9292
DeletionPolicy: Retain
9393
Properties:
9494
AccessControl: LogDeliveryWrite
95+
OwnershipControls:
96+
Rules:
97+
- ObjectOwnership: ObjectWriter
9598
BucketName: !GetAtt GetWebsiteBucketName.Data
9699
BucketEncryption:
97100
ServerSideEncryptionConfiguration:

deployment/content-localization-on-aws.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ Conditions:
2424
Mappings:
2525
MediaInsights:
2626
Release:
27-
Version: "v5.1.0"
27+
Version: "v5.1.1"
2828
Application:
2929
SourceCode:
3030
GlobalS3Bucket: "%%GLOBAL_BUCKET_NAME%%"

deployment/sync-s3-dist.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,8 @@ cd "$build_dir"/ || exit 1
138138
echo "Copying the prepared distribution to:"
139139
echo "s3://$global_bucket/content-localization-on-aws/$version/"
140140
echo "s3://${regional_bucket}-${region}/content-localization-on-aws/$version/"
141+
142+
s3domain="s3.$region.amazonaws.com"
141143
set -x
142144
aws s3 sync $global_dist_dir s3://$global_bucket/content-localization-on-aws/$version/ $(if [ ! -z $profile ]; then echo "--profile $profile"; fi)
143145
aws s3 sync $regional_dist_dir s3://${regional_bucket}-${region}/content-localization-on-aws/$version/ $(if [ ! -z $profile ]; then echo "--profile $profile"; fi)

0 commit comments

Comments
 (0)