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

Commit 9f385c3

Browse files
authored
Merge pull request #740 from aws-solutions/release/v5.1.1
Release/v5.1.1
2 parents 29fdca7 + e73f621 commit 9f385c3

8 files changed

Lines changed: 16 additions & 12 deletions

File tree

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,12 @@ 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+
## [5.1.1] - 2023-04-13
9+
10+
### Changed
11+
12+
* Updated the bucket policy on the DataplaneLogsBucket bucket(s) to grant access to the logging service principal (logging.s3.amazonaws.com) for access log delivery
13+
814
## [5.1.0] - 2023-02-06
915

1016
### Added
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
pymediainfo==4.2.1
22
jsonpickle==3.0.0
3-
aws-xray-sdk==2.11.0
3+
aws-xray-sdk==2.12.0
44
/packages/Media_Insights_Engine_Lambda_Helper-1.0.0-py3-none-any.whl

source/cdk/cdk.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
{
22
"app": "npx ts-node bin/media-insights.ts",
33
"context": {
4+
"@aws-cdk/aws-s3:serverAccessLogsUseBucketPolicy": true,
45
"solution_id": "SO0163",
56
"solution_name": "media-insights-on-aws",
67
"solution_version": "%%VERSION%%"

source/cdk/lib/media-insights-stack.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -334,7 +334,6 @@ export class MediaInsightsStack extends Stack {
334334
const dataplaneLogsBucket = new s3.Bucket(this, 'DataplaneLogsBucket', {
335335
enforceSSL: true,
336336
versioned: true,
337-
accessControl: s3.BucketAccessControl.LOG_DELIVERY_WRITE,
338337
encryption: s3.BucketEncryption.S3_MANAGED,
339338
blockPublicAccess: s3.BlockPublicAccess.BLOCK_ALL,
340339
});
@@ -390,8 +389,6 @@ export class MediaInsightsStack extends Stack {
390389
],
391390
});
392391

393-
dataplaneBucket.node.addDependency(dataplaneLogsBucket);
394-
395392

396393
//
397394
// Service - SNS

source/cdk/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@
3535
"@types/jest": "^27.0.0",
3636
"@types/node": "^16.4.0",
3737
"@types/sinon": "^10.0.2",
38-
"aws-cdk": "^2.51.0",
39-
"aws-cdk-lib": "^2.51.0",
38+
"aws-cdk": "^2.73.0",
39+
"aws-cdk-lib": "^2.73.0",
4040
"constructs": "^10.0.9",
4141
"jest": "^27.0.6",
4242
"sinon": "^11.1.1",
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
simplejson==3.18.0
1+
simplejson==3.19.1
22
jsonpickle==3.0.0
3-
aws-xray-sdk==2.11.0
3+
aws-xray-sdk==2.12.0
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
simplejson==3.18.0
1+
simplejson==3.19.1

test/unit/requirements.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ pytest==7.2.0
44
requests==2.28.1
55
urllib3==1.26.13
66
jsonschema==4.17.0
7-
chalice==1.27.3
8-
aws_xray_sdk==2.11.0
7+
chalice==1.28.0
8+
aws_xray_sdk==2.12.0
99
pytest-cov==4.0.0
10-
simplejson==3.18.0
10+
simplejson==3.19.1
1111
pymediainfo==6.0.1
1212
nltk==3.8.0
1313
webvtt-py==0.4.6

0 commit comments

Comments
 (0)