Skip to content

[27.x backport] gha: govulncheck: make sure read permissions are set#48525

Merged
thaJeztah merged 1 commit intomoby:27.xfrom
thaJeztah:27.x_backport_govulncheck_permissions
Sep 18, 2024
Merged

[27.x backport] gha: govulncheck: make sure read permissions are set#48525
thaJeztah merged 1 commit intomoby:27.xfrom
thaJeztah:27.x_backport_govulncheck_permissions

Conversation

@thaJeztah
Copy link
Member

If any permission is set, any permission not included in the list is implicitly set to "none".

see https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions

The govulncheck check need read permissions, which is not problematic for public repositories, but may be needed when running in a private fork (such as those used for security releases).

(cherry picked from commit 4f1d739)

If any permission is set, any permission not included in the list is
implicitly set to "none".

see https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions

The govulncheck check need read permissions, which is not problematic
for public repositories, but may be needed when running in a private
fork (such as those used for security releases).

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 4f1d739)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah merged commit 41ca978 into moby:27.x Sep 18, 2024
@thaJeztah thaJeztah deleted the 27.x_backport_govulncheck_permissions branch September 18, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants