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