Skip to content

workflow: set restricted permissions#88

Merged
bk2204 merged 1 commit intomainfrom
actions-permissions
Oct 22, 2024
Merged

workflow: set restricted permissions#88
bk2204 merged 1 commit intomainfrom
actions-permissions

Conversation

@bk2204
Copy link
Copy Markdown
Contributor

@bk2204 bk2204 commented Oct 22, 2024

If no explicit permissions are set for a GitHub Actions run, then the repository or organization permissions are used, which can be too broad. Let's specifically set the permissions to read-only for the contents of the repository, since we only need to clone and test our code. That way, we'll be taking advantage of the principle of least privilege.

If no explicit permissions are set for a GitHub Actions run, then the
repository or organization permissions are used, which can be too broad.
Let's specifically set the permissions to read-only for the contents of
the repository, since we only need to clone and test our code. That way,
we'll be taking advantage of the principle of least privilege.
@bk2204 bk2204 marked this pull request as ready for review October 22, 2024 17:56
@bk2204 bk2204 requested a review from a team October 22, 2024 17:56
@bk2204 bk2204 merged commit a33b61c into main Oct 22, 2024
@bk2204 bk2204 deleted the actions-permissions branch October 22, 2024 20:38
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