Skip to content

Update README.md#348

Merged
tsusdere merged 6 commits intomainfrom
oidc-doc
May 31, 2024
Merged

Update README.md#348
tsusdere merged 6 commits intomainfrom
oidc-doc

Conversation

@tsusdere
Copy link
Copy Markdown
Contributor

Adds information about our GitHub generated OICD tokens.

@tsusdere tsusdere requested a review from a team as a code owner May 22, 2024 18:56
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@tsusdere tsusdere requested a review from GretaP May 24, 2024 18:16
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
GretaP
GretaP previously approved these changes May 31, 2024
Comment thread README.md Outdated
Co-authored-by: Mingzi  <yimysty@github.com>
@tsusdere tsusdere merged commit 199a735 into main May 31, 2024
@tsusdere tsusdere deleted the oidc-doc branch May 31, 2024 19:48
Comment thread README.md
OIDC tokens are minted within the context of a single job, and are used to form a trust relationship which validates properties of the workflow run against a third-party (e.g. cloud providers such as AWS or Azure). In the context of GitHub Pages, this is most relevant to ensure a workflow respects branch protection settings. To do this, the OIDC token includes a claim about which branch/ref is executing the workflow. The token is passed to the pages deployment API as part of the request payload, where it's decoded internally to validate the claims and verify if that workflow is allowed to deploy to pages.
A common question regarding OIDC tokens is the use of `pages:write` and `id-token:write`. Ideally, the `GITHUB_TOKEN` would contain both but currently our API for tokens does not operate this way hence the need for
A common question regarding OIDC tokens is the need to use both `pages:write` and `id-token:write`. The pages permission relates to the `GITHUB_TOKEN` by giving it the permissions to create pages deployments when calling the GitHub API. The id-token permission is necessary to request the OIDC JWT token. For more information on the id-token, check the docs on [adding permissions settings](https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-cloud-providers#adding-permissions-settings)
A common question regarding OIDC tokens is the need to use both `pages:write` and `id-token:write`. The pages permission relates to the `GITHUB_TOKEN` by giving it the permissions to create pages deployments when calling the GitHub API. The id-token permission is necessary to request the OIDC JWT token. For more information on the id-token, check the docs on [adding permissions settings](https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-cloud-providers#adding-permissions-settings)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which of these last 3 paragraphs did we mean to include?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants