Skip to content

[sec-check] fix: unit-tests.yml add permissions block (contents: read) - #961

Open
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
sec/fix-unit-tests-permissions
Open

[sec-check] fix: unit-tests.yml add permissions block (contents: read)#961
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
sec/fix-unit-tests-permissions

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Security Fix

Adds permissions: contents: read to .github/workflows/unit-tests.yml. The workflow runs on pull_request, push, merge_group, and workflow_dispatch with no declared permissions, so GITHUB_TOKEN inherited the repository default. The test job is read-only (artifact upload works with contents: read).

Fixes #954


Filed by sec-check agent (ACMM L4/L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=sec-check backend=copilot model=kimi-k3 copilot=1.0.59

Workflow runs on pull_request, push, merge_group, and workflow_dispatch
with no declared permissions, so GITHUB_TOKEN inherits the repository
default. The test job is read-only (shellcheck, pytest, bats; artifact
upload works with contents: read).

Fixes #954

Signed-off-by: kubestellar-hive[bot] <280983584+kubestellar-hive[bot]@users.noreply.github.com>
@kubestellar-hive kubestellar-hive Bot added hold Work is intentionally paused. security labels Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Work is intentionally paused. security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[sec-check] unit-tests.yml: missing permissions block on PR-triggered workflow

0 participants