Skip to content

chore: Set permissions for GitHub actions - #15300

Merged
rolfbjarne merged 1 commit into
dotnet:mainfrom
turrisxyz:Pinned-Dependencies-GitHub
Jun 22, 2022
Merged

chore: Set permissions for GitHub actions#15300
rolfbjarne merged 1 commit into
dotnet:mainfrom
turrisxyz:Pinned-Dependencies-GitHub

Conversation

@nathannaveen

Copy link
Copy Markdown
Contributor

 Restrict the GitHub token permissions only to the required ones; this way, even if the attackers will succeed in compromising your workflow, they won’t be able to do much.

- Included permissions for the action. https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions

https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs

[Keeping your GitHub Actions and workflows secure Part 1: Preventing pwn requests](https://securitylab.github.com/research/github-actions-preventing-pwn-requests/)

Signed-off-by: nathannaveen <42319948+nathannaveen@users.noreply.github.com>
@net-foundation-cla

net-foundation-cla Bot commented Jun 20, 2022

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@mandel-macaque

Copy link
Copy Markdown
Contributor

@nathannaveen we need you to sing the CLA or we won't be allows to land the PR.

@mandel-macaque mandel-macaque added community Community contribution âĪ do-not-merge Do not merge this pull request labels Jun 20, 2022
@nathannaveen

Copy link
Copy Markdown
Contributor Author

@nathannaveen we need you to sing the CLA or we won't be allows to land the PR.

Thanks, I have done that

@rolfbjarne
rolfbjarne merged commit b03bd25 into dotnet:main Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Community contribution âĪ do-not-merge Do not merge this pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants