-
Notifications
You must be signed in to change notification settings - Fork 505
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
Scorecard repo fails at Token-Permissions
with score 0
#942
Comments
@naveensrinivasan , can you handle it? |
As we fix this, let's keep a record to explain the permissions we need. Everyone struggles with GitHub permissions AFAIK |
I will take this. |
It would be great to have a tool that will produce the actual permissions needed for these actions. |
* Updated the actions permissions for codeql from write to specific settings. #942
* Updated the actions permissions for codeql from write to specific settings. #942
* Updated the actions permissions for codeql from write to specific settings. #942
Can we close this issue now? @laurentsimon @naveensrinivasan |
yes. Scorecard still gives a score of 0 but it's a bug. Will create an issue for tracking #1400 |
We need to fix this. A cursory read of our workflows:
contents:read
,statuses: write
and possiblysecurity-events: write
contents:read
contents:read
,pull-requests:write
for this.contents:read
,status:write
for license part.contents:read
andissues:write
andpull-requests:write
contents:read
andstatuses:write
We can define
permission:
to set all permissions tonone
at the start of the workflows, and then set what we want in eachruns
. It's fine to setcontents:read
at the start of workflows, if this helps.See https://github.blog/changelog/2021-04-20-github-actions-control-permissions-for-github_token/ for more info
The text was updated successfully, but these errors were encountered: