Update integration test, add feedback from Jimmy

aws · Oct 14, 2024 · 40cc995 · 40cc995
1 parent 2a1fe8f
commit 40cc995
Show file tree

Hide file tree

Showing 14 changed files with 1,176 additions and 208 deletions.
diff --git a/packages/@aws-cdk/aws-msk-alpha/lib/cluster.ts b/packages/@aws-cdk/aws-msk-alpha/lib/cluster.ts
@@ -654,16 +654,16 @@ export class Cluster extends ClusterBase {
     }
 
     let clientAuthentication;
+    const { saslProps, tlsProps } = props.clientAuthentication ?? {};
+
     if (props.clientAuthentication) {
       clientAuthentication = {
-        sasl: props.clientAuthentication.saslProps ? {
-          iam: props.clientAuthentication.saslProps.iam ? { enabled: true }: undefined,
-          scram: props.clientAuthentication.saslProps.scram ? { enabled: true }: undefined,
+        sasl: saslProps ? {
+          iam: saslProps.iam ? { enabled: true }: undefined,
+          scram: saslProps.scram ? { enabled: true }: undefined,
         } : undefined,
-        tls: props.clientAuthentication.tlsProps?.certificateAuthorities ? {
-          certificateAuthorityArnList: props.clientAuthentication.tlsProps.certificateAuthorities?.map(
-            (ca) => ca.certificateAuthorityArn,
-          ),
+        tls: tlsProps?.certificateAuthorities ? {
+          certificateAuthorityArnList: tlsProps.certificateAuthorities?.map((ca) => ca.certificateAuthorityArn),
           enabled: true,
         } : undefined,
       };