Skip to content

Set permissions for GitHub actions#1432

Merged
bitwiseman merged 1 commit into
hub4j:mainfrom
turrisxyz:naveen/feat/set-perms-actions
Apr 15, 2022
Merged

Set permissions for GitHub actions#1432
bitwiseman merged 1 commit into
hub4j:mainfrom
turrisxyz:naveen/feat/set-perms-actions

Conversation

@naveensrinivasan
Copy link
Copy Markdown
Contributor

- 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/)

 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.

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 15, 2022

Codecov Report

Merging #1432 (740ed06) into main (cb7aacf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1432   +/-   ##
=========================================
  Coverage     78.62%   78.62%           
  Complexity     2090     2090           
=========================================
  Files           201      201           
  Lines          6381     6381           
  Branches        357      357           
=========================================
  Hits           5017     5017           
  Misses         1157     1157           
  Partials        207      207           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb7aacf...740ed06. Read the comment docs.

@bitwiseman bitwiseman merged commit 2cab124 into hub4j:main Apr 15, 2022
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