fix(iam): allow CompositePrincipal construction with spread (#2507)

aws · May 10, 2019 · eb13741 · eb13741
1 parent 1feda0c
commit eb13741
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/packages/@aws-cdk/aws-iam/lib/policy-document.ts b/packages/@aws-cdk/aws-iam/lib/policy-document.ts
@@ -287,11 +287,13 @@ export class CompositePrincipal extends PrincipalBase {
   public readonly assumeRoleAction: string;
   private readonly principals = new Array<PrincipalBase>();
 
-  constructor(principal: PrincipalBase, ...additionalPrincipals: PrincipalBase[]) {
+  constructor(...principals: PrincipalBase[]) {
     super();
-    this.assumeRoleAction = principal.assumeRoleAction;
-    this.addPrincipals(principal);
-    this.addPrincipals(...additionalPrincipals);
+    if (principals.length === 0) {
+      throw new Error('CompositePrincipals must be constructed with at least 1 Principal but none were passed.');
+    }
+    this.assumeRoleAction = principals[0].assumeRoleAction;
+    this.addPrincipals(...principals);
   }
 
   public addPrincipals(...principals: PrincipalBase[]): this {