Skip to content

[CI] Add permissions to workflow#11

Merged
TomerShor merged 1 commit intonuclio:masterfrom
TomerShor:action-permissions
Dec 14, 2025
Merged

[CI] Add permissions to workflow#11
TomerShor merged 1 commit intonuclio:masterfrom
TomerShor:action-permissions

Conversation

@TomerShor
Copy link
Contributor

@TomerShor TomerShor commented Dec 14, 2025

Added the the content: read and actions: write permissions.

Why actions: write is needed:

  1. actions/cache@v5 — Requires write access to save and restore caches
  2. actions/setup-go@v6 with cache: true — Uses the cache action internally, so it also needs write access

Current permissions:

  • contents: read — To checkout the repository code
  • actions: write — To save/restore caches for faster builds

This follows the principle of least privilege: only the permissions needed for the workflow to function. Without actions: write, the cache actions will fail silently or with permission errors.

@TomerShor TomerShor merged commit f9edffb into nuclio:master Dec 14, 2025
2 checks passed
@TomerShor TomerShor deleted the action-permissions branch December 14, 2025 11:29
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.

1 participant