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

Build: Set minimum permission for GitHub Actions Token #2129

Merged
merged 1 commit into from Dec 1, 2022

Conversation

boahc077
Copy link
Contributor

@boahc077 boahc077 commented Oct 31, 2022

The following GitHub Actions workflow execution logs demonstrates that the GitHub Action token permissions are elevated:
https://github.com/jquery/jquery-ui/actions/runs/3218073141/jobs/5261823755#step:1:19

This PR fixes the issue.

In addition to this PR, if you have repo admin access, then you should consider setting the following permission so that new workflow files will default to read-only permissions
https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#setting-the-permissions-of-the-github_token-for-your-repository

Signed-off-by: Ashish Kurmi akurmi@stepsecurity.io

Signed-off-by: Ashish Kurmi <akurmi@stepsecurity.io>
mgol
mgol approved these changes Dec 1, 2022
Copy link
Member

@mgol mgol left a comment

Thank you!

@mgol mgol added this to the 1.13.3 milestone Dec 1, 2022
@mgol mgol merged commit 970a0cf into jquery:main Dec 1, 2022
4 checks passed
@mgol
Copy link
Member

mgol commented Dec 1, 2022

I also set the default permissions to read-only for jQuery, jQuery UI, jQuery Migrate and also at the jQuery organization level.

Thanks for the heads-up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants