fix(logs): DataProtectionPolicy template key must be PascalCase (#3…

…3462) ### Issue # (if applicable) Closes #33408 Closes #26728 ### Reason for this change Invalid DataProtectionPolicy syntax ### Description of changes Change template key from camelCase > PascalCase ### Description of how you validated changes Unit + integration ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Feb 19, 2025 · 0379878 · 0379878
1 parent 7901f07
commit 0379878
Show file tree

Hide file tree

Showing 11 changed files with 214 additions and 203 deletions.
diff --git a/...test/integ.log-group.js.snapshot/LogGroupIntegDefaultTestDeployAssertA9999A13.assets.json b/...test/integ.log-group.js.snapshot/LogGroupIntegDefaultTestDeployAssertA9999A13.assets.json
diff --git a/...-integ/test/aws-logs/test/integ.log-group.js.snapshot/aws-cdk-log-group-integ.assets.json b/...-integ/test/aws-logs/test/integ.log-group.js.snapshot/aws-cdk-log-group-integ.assets.json
diff --git a/...nteg/test/aws-logs/test/integ.log-group.js.snapshot/aws-cdk-log-group-integ.template.json b/...nteg/test/aws-logs/test/integ.log-group.js.snapshot/aws-cdk-log-group-integ.template.json
@@ -17,21 +17,13 @@
    "Type": "AWS::Logs::LogGroup",
    "Properties": {
     "DataProtectionPolicy": {
-     "name": "policy-name",
-     "description": "policy description",
-     "version": "2021-06-01",
-     "configuration": {
-      "customDataIdentifier": [
-       {
-        "name": "EmployeeId",
-        "regex": "EmployeeId-\\d{9}"
-       }
-      ]
-     },
-     "statement": [
+     "Name": "policy-name",
+     "Description": "policy description",
+     "Version": "2021-06-01",
+     "Statement": [
       {
-       "sid": "audit-statement-cdk",
-       "dataIdentifier": [
+       "Sid": "audit-statement-cdk",
+       "DataIdentifier": [
         {
          "Fn::Join": [
           "",
@@ -58,16 +50,16 @@
         },
         "EmployeeId"
        ],
-       "operation": {
-        "audit": {
-         "findingsDestination": {
-          "cloudWatchLogs": {
-           "logGroup": {
+       "Operation": {
+        "Audit": {
+         "FindingsDestination": {
+          "CloudWatchLogs": {
+           "LogGroup": {
             "Ref": "LogGroupLambdaAuditF8F47F46"
            }
           },
-          "s3": {
-           "bucket": {
+          "S3": {
+           "Bucket": {
             "Ref": "auditbucketidE6660EBD"
            }
           }
@@ -76,8 +68,8 @@
        }
       },
       {
-       "sid": "redact-statement-cdk",
-       "dataIdentifier": [
+       "Sid": "redact-statement-cdk",
+       "DataIdentifier": [
         {
          "Fn::Join": [
           "",
@@ -104,13 +96,21 @@
         },
         "EmployeeId"
        ],
-       "operation": {
-        "deidentify": {
-         "maskConfig": {}
+       "Operation": {
+        "Deidentify": {
+         "MaskConfig": {}
         }
        }
       }
-     ]
+     ],
+     "Configuration": {
+      "CustomDataIdentifier": [
+       {
+        "Name": "EmployeeId",
+        "Regex": "EmployeeId-\\d{9}"
+       }
+      ]
+     }
     },
     "RetentionInDays": 731
    },

diff --git a/...s/@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/cdk.out b/...s/@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/cdk.out
diff --git a/...aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/integ.json b/...aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/integ.json
diff --git a/...-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/manifest.json b/...-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/manifest.json
diff --git a/...@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/tree.json b/...@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.js.snapshot/tree.json
diff --git a/packages/@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.ts b/packages/@aws-cdk-testing/framework-integ/test/aws-logs/test/integ.log-group.ts
@@ -28,4 +28,3 @@ class LogGroupIntegStack extends Stack {
 const app = new App();
 const stack = new LogGroupIntegStack(app, 'aws-cdk-log-group-integ');
 new IntegTest(app, 'LogGroupInteg', { testCases: [stack] });
-app.synth();