Skip to content

Commit

Permalink
Merge pull request #563 from russau/docs/readme-role-update
Browse files Browse the repository at this point in the history
Minor: include a role condition test for AUD in sample CFN template
  • Loading branch information
peterwoodworth committed Feb 22, 2023
2 parents 32eb1b4 + 85ec61b commit 567d414
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -161,6 +161,10 @@ Parameters:
Description: Arn for the GitHub OIDC Provider.
Default: ""
Type: String
OIDCAudience:
Description: Audience supplied to configure-aws-credentials.
Default: "sts.amazonaws.com"
Type: String

Conditions:
CreateOIDCProvider: !Equals
Expand All @@ -181,6 +185,8 @@ Resources:
- !Ref GithubOidc
- !Ref OIDCProviderArn
Condition:
StringEquals:
token.actions.githubusercontent.com:aud: !Ref OIDCAudience
StringLike:
token.actions.githubusercontent.com:sub: !Sub repo:${GitHubOrg}/${RepositoryName}:*

Expand Down

0 comments on commit 567d414

Please sign in to comment.