Skip to content

ci(suite): cap GITHUB_TOKEN to contents: read - #92

Merged
tisonkun merged 2 commits into
pingcap:masterfrom
arpitjain099:chore/declare-workflow-perms
Aug 11, 2026
Merged

ci(suite): cap GITHUB_TOKEN to contents: read#92
tisonkun merged 2 commits into
pingcap:masterfrom
arpitjain099:chore/declare-workflow-perms

Conversation

@arpitjain099

Copy link
Copy Markdown
Contributor

The Build & Test workflow runs make build test only. No GitHub API writes, so a workflow-level contents: read is the appropriate ceiling for the default GITHUB_TOKEN.

Same post-CVE-2025-30066 supply-chain hardening pattern (tj-actions/changed-files). YAML validated locally.

Build & test workflow runs make build/test only; no GitHub API writes. contents: read at workflow level is the appropriate cap.

Post-CVE-2025-30066 hardening shape. yaml.safe_load validated.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@pingcap-cla-assistant

pingcap-cla-assistant Bot commented May 21, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@tisonkun tisonkun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@tisonkun
tisonkun merged commit 55ac33a into pingcap:master Aug 11, 2026
2 checks passed
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.

2 participants