Skip to content

Commit

Permalink
Merge branch 'main' into od-lowest-price
Browse files Browse the repository at this point in the history
  • Loading branch information
paulhcsun committed Dec 18, 2023
2 parents da9d374 + 49358fd commit 5801a0d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions packages/aws-cdk-lib/aws-apigateway/lib/authorizers/lambda.ts
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ abstract class LambdaAuthorizer extends Authorizer implements IAuthorizer {
*/
protected setupPermissions() {
if (!this.role) {
this.addDefaultPermisionRole();
this.addDefaultPermissionRole();
} else if (iam.Role.isRole(this.role)) {
this.addLambdaInvokePermission(this.role);
}
Expand All @@ -132,17 +132,17 @@ abstract class LambdaAuthorizer extends Authorizer implements IAuthorizer {
/**
* Add Default Permission Role for handler
*/
private addDefaultPermisionRole() :void {
private addDefaultPermissionRole(): void {
this.handler.addPermission(`${Names.uniqueId(this)}:Permissions`, {
principal: new iam.ServicePrincipal('apigateway.amazonaws.com'),
sourceArn: this.authorizerArn,
});
}

/**
* Add Lambda Invoke Permission for LambdaAurhorizer's role
* Add Lambda Invoke Permission for Lambda Authorizer's role
*/
private addLambdaInvokePermission(role: iam.Role) :void {
private addLambdaInvokePermission(role: iam.Role): void {
role.attachInlinePolicy(new iam.Policy(this, 'authorizerInvokePolicy', {
statements: [
new iam.PolicyStatement({
Expand Down

0 comments on commit 5801a0d

Please sign in to comment.