Skip to content

chore: Set permissions for GitHub actions#1901

Merged
kataras merged 1 commit intokataras:masterfrom
turrisxyz:Pinned-Dependencies-GitHub
Jun 1, 2022
Merged

chore: Set permissions for GitHub actions#1901
kataras merged 1 commit intokataras:masterfrom
turrisxyz:Pinned-Dependencies-GitHub

Conversation

@naveensrinivasan
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: naveen <172697+naveensrinivasan@users.noreply.github.com>
@naveensrinivasan naveensrinivasan requested a review from kataras as a code owner June 1, 2022 00:29
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jun 1, 2022

CLA assistant check
All committers have signed the CLA.

@kataras
Copy link
Copy Markdown
Owner

kataras commented Jun 1, 2022

Hello @naveensrinivasan, thanks a lot. Could you please accept the CLA so I can approve & merge this PR?

@naveensrinivasan
Copy link
Copy Markdown
Contributor Author

Hello @naveensrinivasan, thanks a lot. Could you please accept the CLA so I can approve & merge this PR?

Thanks. I have signed it.

@kataras kataras merged commit ef8799f into kataras:master Jun 1, 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.

3 participants