Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Set permissions for GitHub actions #5523

Merged

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>
@naveensrinivasan naveensrinivasan requested a review from a team as a code owner June 30, 2022 00:29
@github-actions github-actions bot added the github_actions Pull requests that update Github_actions code label Jun 30, 2022
@eddumelendez
Copy link
Member

eddumelendez commented Jun 30, 2022

@naveensrinivasan thanks for the PR! just one question. do we also need the permission content:read here, right?

@kiview kiview added this to the next milestone Jul 1, 2022
@naveensrinivasan
Copy link
Contributor Author

@naveensrinivasan thanks for the PR! just one question. do we also need the permission content:read here, right?

I think so.

@kiview
Copy link
Member

kiview commented Jul 5, 2022

@naveensrinivasan Would you also like to contribute this change, so we have everything together in this PR? 🙂

@naveensrinivasan
Copy link
Contributor Author

@naveensrinivasan Would you also like to contribute this change, so we have everything together in this PR? 🙂

Apologies I won’t be able to do it. Thanks

@eddumelendez eddumelendez merged commit 3641114 into testcontainers:master Jul 7, 2022
@eddumelendez
Copy link
Member

thanks for your contribution @naveensrinivasan ! this is now merged in master branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update Github_actions code type/housekeeping
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants