Skip to content

Commit ef6dd32

Browse files
committed
fix(iam): throw error when statement id is invalid (#34819)
1 parent 6051039 commit ef6dd32

File tree

34 files changed

+1292
-76
lines changed

34 files changed

+1292
-76
lines changed

packages/@aws-cdk-testing/framework-integ/test/aws-ecs/test/integ.cluster-encrypt-ephemeral-storage.js.snapshot/aws-ecs-ephemeral-integ.template.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@
435435
"Service": "fargate.amazonaws.com"
436436
},
437437
"Resource": "*",
438-
"Sid": "Allow generate data key access for Fargate tasks."
438+
"Sid": "AllowGenerateDataKeyAccessForFargateTasks"
439439
},
440440
{
441441
"Action": "kms:CreateGrant",
@@ -458,7 +458,7 @@
458458
"Service": "fargate.amazonaws.com"
459459
},
460460
"Resource": "*",
461-
"Sid": "Allow grant creation permission for Fargate tasks."
461+
"Sid": "AllowGrantCreationPermissionForFargateTasks"
462462
}
463463
],
464464
"Version": "2012-10-17"
@@ -526,7 +526,7 @@
526526
"Service": "fargate.amazonaws.com"
527527
},
528528
"Resource": "*",
529-
"Sid": "Allow generate data key access for Fargate tasks."
529+
"Sid": "AllowGenerateDataKeyAccessForFargateTasks"
530530
},
531531
{
532532
"Action": "kms:CreateGrant",
@@ -552,7 +552,7 @@
552552
"Service": "fargate.amazonaws.com"
553553
},
554554
"Resource": "*",
555-
"Sid": "Allow grant creation permission for Fargate tasks."
555+
"Sid": "AllowGrantCreationPermissionForFargateTasks"
556556
}
557557
],
558558
"Version": "2012-10-17"

packages/@aws-cdk-testing/framework-integ/test/aws-ecs/test/integ.cluster-encrypt-ephemeral-storage.js.snapshot/tree.json

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-events/test/integ.eventbus-cross-account-grants.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ const originalBus = new EventBus(crossAccountStack, 'bus', {
103103
});
104104
testingStack.addDependency(crossAccountStack);
105105

106-
originalBus.grantPutEventsTo(new AccountPrincipal(account), 'cross-account-access-sid');
106+
originalBus.grantPutEventsTo(new AccountPrincipal(account), 'crossAccountAccessSid');
107107

108108
const integ = new IntegTest(app, 'CrossAccountDeploy', {
109109
testCases: [

packages/@aws-cdk-testing/framework-integ/test/aws-events/test/integ.eventbus.js.snapshot/Stack.template.json

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
"Name": "StackBusAA0A1E4B"
4444
}
4545
},
46-
"BuscdkStatement1D7D87B9D": {
46+
"BuscdkStatement17E48E06E": {
4747
"Type": "AWS::Events::EventBusPolicy",
4848
"Properties": {
4949
"EventBusName": {
@@ -66,12 +66,12 @@
6666
"Arn"
6767
]
6868
},
69-
"Sid": "cdk-Statement1"
69+
"Sid": "cdkStatement1"
7070
},
71-
"StatementId": "cdk-Statement1"
71+
"StatementId": "cdkStatement1"
7272
}
7373
},
74-
"BuscdkStatement2341A5B58": {
74+
"BuscdkStatement2D6A35B77": {
7575
"Type": "AWS::Events::EventBusPolicy",
7676
"Properties": {
7777
"EventBusName": {
@@ -94,9 +94,9 @@
9494
"Arn"
9595
]
9696
},
97-
"Sid": "cdk-Statement2"
97+
"Sid": "cdkStatement2"
9898
},
99-
"StatementId": "cdk-Statement2"
99+
"StatementId": "cdkStatement2"
100100
}
101101
}
102102
},

packages/@aws-cdk-testing/framework-integ/test/aws-events/test/integ.eventbus.js.snapshot/manifest.json

Lines changed: 4 additions & 24 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)