mirror of
https://github.com/fatedier/frp.git
synced 2026-05-15 08:05:49 -06:00
[PR #3155] [MERGED] Set least privileged token permission for GitHub Actions #4691
Labels
No labels
In Progress
WIP
WaitingForInfo
bug
doc
duplicate
easy
enhancement
future
help wanted
invalid
lifecycle/stale
need-issue-template
need-usage-help
no plan
proposal
pull-request
question
todo
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/frp#4691
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/fatedier/frp/pull/3155
Author: @ashishkurmi
Created: 10/31/2022
Status: ✅ Merged
Merged: 10/31/2022
Merged by: @fatedier
Base:
dev← Head:dev📝 Commits (1)
85397edSet least privileged token permission for GitHub Actions📊 Changes
3 files changed (+12 additions, -0 deletions)
View changed files
📝
.github/workflows/build-and-push-image.yml(+3 -0)📝
.github/workflows/goreleaser.yml(+3 -0)📝
.github/workflows/stale.yml(+6 -0)📄 Description
Other than
golangci-lint.yml, all other GitHub Actions workflow files have elevated token permissions as demonstrated by the following workflow execution logs.https://github.com/fatedier/frp/actions/runs/3334432128/jobs/5517384845#step:1:19
This PR mitigates this issue. In addition to this PR, 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
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.