Skip to content

chore: Set permissions for GitHub actions#1422

Merged
sl0thentr0py merged 2 commits intogetsentry:masterfrom
turrisxyz:setup-permissions
May 4, 2022
Merged

chore: Set permissions for GitHub actions#1422
sl0thentr0py merged 2 commits intogetsentry:masterfrom
turrisxyz:setup-permissions

Conversation

@naveensrinivasan
Copy link
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: naveen <172697+naveensrinivasan@users.noreply.github.com>
@sl0thentr0py sl0thentr0py enabled auto-merge (squash) May 4, 2022 19:49
Copy link
Member

@sl0thentr0py sl0thentr0py left a comment

Choose a reason for hiding this comment

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

ty for the contribution!

@sl0thentr0py sl0thentr0py merged commit 0b32de6 into getsentry:master May 4, 2022
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.

2 participants