chore(logs): fix method name typo (#28416)

Fix typo in method name (`convertArnPrincpalToAccountId` -> `convertArnPrincipalToAccountId`) and another `princpal` typo. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Dec 18, 2023 · 0e30fcc · 0e30fcc
1 parent 0b2ba1c
commit 0e30fcc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/aws-cdk-lib/aws-iam/lib/principals.ts b/packages/aws-cdk-lib/aws-iam/lib/principals.ts
@@ -855,7 +855,7 @@ export class CompositePrincipal extends PrincipalBase {
 
   public get policyFragment(): PrincipalPolicyFragment {
     // We only have a problem with conditions if we are trying to render composite
-    // princpals into a single statement (which is when `policyFragment` would get called)
+    // principals into a single statement (which is when `policyFragment` would get called)
     for (const p of this._principals) {
       const fragment = p.policyFragment;
       if (fragment.conditions && Object.keys(fragment.conditions).length > 0) {

diff --git a/packages/aws-cdk-lib/aws-logs/lib/log-group.ts b/packages/aws-cdk-lib/aws-logs/lib/log-group.ts
@@ -223,12 +223,12 @@ abstract class LogGroupBase extends Resource implements ILogGroup {
       this.policy = new ResourcePolicy(this, 'Policy');
     }
     this.policy.document.addStatements(statement.copy({
-      principals: statement.principals.map(p => this.convertArnPrincpalToAccountId(p)),
+      principals: statement.principals.map(p => this.convertArnPrincipalToAccountId(p)),
     }));
     return { statementAdded: true, policyDependable: this.policy };
   }
 
-  private convertArnPrincpalToAccountId(principal: iam.IPrincipal) {
+  private convertArnPrincipalToAccountId(principal: iam.IPrincipal) {
     if (principal.principalAccount) {
       // we use ArnPrincipal here because the constructor inserts the argument
       // into the template without mutating it, which means that there is no