Skip to content

Commit

Permalink
fix: removes names from IAM policy and role, as this conflicts with m…
Browse files Browse the repository at this point in the history
…ulti-region deployments (#184)

* fix: removes names from IAM policy and role, as this conflicts with multi-region deployments

* fix: removes names from IAM policy and role, as this conflicts with multi-region deployments

* fix: removes names from IAM policy and role, as this conflicts with multi-region deployments
  • Loading branch information
udondan committed Oct 13, 2023
1 parent 4ae631f commit 165168f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion VERSION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3.3.2
3.3.3
2 changes: 0 additions & 2 deletions lib/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,6 @@ export class KeyPair extends Construct implements ITaggable {
stack,
'EC2-Key-Pair-Manager-Policy',
{
managedPolicyName: `${this.prefix}-${cleanID}`,
description: `Used by Lambda ${cleanID}, which is a custom CFN resource, managing EC2 Key Pairs`,
statements: [
new aws_iam.PolicyStatement({
Expand Down Expand Up @@ -355,7 +354,6 @@ export class KeyPair extends Construct implements ITaggable {
);

const role = new aws_iam.Role(stack, 'EC2-Key-Pair-Manager-Role', {
roleName: `${this.prefix}-${cleanID}`,
description: `Used by Lambda ${cleanID}, which is a custom CFN resource, managing EC2 Key Pairs`,
assumedBy: new aws_iam.ServicePrincipal('lambda.amazonaws.com'),
managedPolicies: [
Expand Down

0 comments on commit 165168f

Please sign in to comment.