Skip to content

Commit 223f8ba

Browse files
authored
Merge pull request #960 from dan1elt0m/fix-rights-issue-secrets-manager
Fix secrets manager rights issue
2 parents 4a80222 + 827b6ec commit 223f8ba

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

typescript/src/resources/deploy-lambda.ts

+8-2
Original file line numberDiff line numberDiff line change
@@ -298,9 +298,15 @@ export class DatabricksDeployLambda extends IDatabricksDeployLambda {
298298

299299
this.lambdaRole.addToPrincipalPolicy(new aws_iam.PolicyStatement({
300300
effect: aws_iam.Effect.ALLOW,
301-
actions: ["secretsmanager:CreateSecret", "secretsmanager:ListSecrets", "secretsmanager:DeleteSecret"],
301+
actions: [ "secretsmanager:ListSecrets"],
302+
resources: ["*"] // AWS doesn't support providing specific resources for the ListSecrets action
303+
}));
304+
305+
this.lambdaRole.addToPrincipalPolicy(new aws_iam.PolicyStatement({
306+
effect: aws_iam.Effect.ALLOW,
307+
actions: ["secretsmanager:CreateSecret", "secretsmanager:DeleteSecret"],
302308
resources: [
303-
`arn:aws:ssm:${this.props.region}:${this.props.accountId}:secret/databricks/token/*`,
309+
`arn:aws:secretsmanager:${this.props.region}:${this.props.accountId}:secret:/databricks/token/*`,
304310
]
305311
}));
306312

0 commit comments

Comments
 (0)