Skip to content

🦅 [gha] improve security#52

Merged
chicks-net merged 3 commits intomainfrom
chicks/2025-09-07-gha-permissions
Sep 7, 2025
Merged

🦅 [gha] improve security#52
chicks-net merged 3 commits intomainfrom
chicks/2025-09-07-gha-permissions

Conversation

@chicks-net
Copy link
Member

@chicks-net chicks-net commented Sep 7, 2025

Context

fini-net/fini-infra#15

Done:

  • 🦅 [gha] improve security

(Automated in justfile.)

Copilot AI review requested due to automatic review settings September 7, 2025 10:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves GitHub Actions security by implementing the principle of least privilege for workflow permissions. The changes replace global read-all permissions with minimal scoped permissions at the job level.

  • Removes global read-all permissions and sets global permissions to empty
  • Adds specific job-level permissions with minimal required scopes
  • Ensures each workflow only has access to the resources it actually needs

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/markdownlint.yml Restricts permissions to only contents: read for linting markdown files
.github/workflows/checkov.yml Removes global permissions, keeps existing job-level security scanning permissions
.github/workflows/auto-assign.yml Adds contents: read permission to existing issue/PR write permissions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@chicks-net chicks-net merged commit 8fc4be1 into main Sep 7, 2025
4 checks passed
@chicks-net chicks-net deleted the chicks/2025-09-07-gha-permissions branch September 7, 2025 10: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