Skip to content

chore: Set permissions for GitHub actions - #6467

Merged
yadvr merged 1 commit into
apache:mainfrom
turrisxyz:Pinned-Dependencies-GitHub
Jul 1, 2022
Merged

chore: Set permissions for GitHub actions#6467
yadvr merged 1 commit into
apache:mainfrom
turrisxyz:Pinned-Dependencies-GitHub

Conversation

@neilnaveen

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: neilnaveen <42328488+neilnaveen@users.noreply.github.com>
@boring-cyborg

boring-cyborg Bot commented Jun 17, 2022

Copy link
Copy Markdown

Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
Here are some useful points:

@yadvr yadvr added this to the 4.18.0.0 milestone Jun 21, 2022
@yadvr
yadvr merged commit 15ab481 into apache:main Jul 1, 2022
@boring-cyborg

boring-cyborg Bot commented Jul 1, 2022

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request!

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