Skip to content

[sec-check] fix: declare top-level permissions: {} in e2e.yml pr-e2e.yml promotion-candidate-e2e.yml - #969

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

[sec-check] fix: declare top-level permissions: {} in e2e.yml pr-e2e.yml promotion-candidate-e2e.yml #969
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
sec/fix-top-level-permissions

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Security Fix

Adds an explicit top-level permissions: {} to: e2e.yml pr-e2e.yml promotion-candidate-e2e.yml

Each workflow already declares least-privilege permissions at the job level (verified: every job has its own block). The top-level nil default means any future job added without a permissions block gets an empty token instead of inheriting the repository default (potentially write).

Fixes #962
Fixes #964
Fixes #965


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

…yml promotion-candidate-e2e.yml

Each workflow already declares least-privilege permissions at the job level;
this makes the default explicit so any future job added without a permissions
block gets a nil token instead of the repository default.

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 8, 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

0 participants