Skip to content

Commit

Permalink
fix: don't allow dashes in sids
Browse files Browse the repository at this point in the history
  • Loading branch information
jakeyheath authored May 1, 2024
1 parent cebf994 commit 7959ea0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion aws-iam-role-github-action/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ data "aws_iam_policy_document" "assume_role" {
for_each = var.authorized_github_repos

content {
sid = "AllowGithubActionsToAssumeRole${statement.key}"
sid = replace("Allow${statement.key}ToAssumeRole", "-", "")
principals {
type = "Federated"
identifiers = [local.idp_arn]
Expand Down

0 comments on commit 7959ea0

Please sign in to comment.