Skip to content

ci: declare workflow-level contents: read on unit-test and ci-pull_request#6444

Closed
arpitjain099 wants to merge 1 commit into
elastic:mainfrom
arpitjain099:chore/declare-workflow-perms
Closed

ci: declare workflow-level contents: read on unit-test and ci-pull_request#6444
arpitjain099 wants to merge 1 commit into
elastic:mainfrom
arpitjain099:chore/declare-workflow-perms

Conversation

@arpitjain099
Copy link
Copy Markdown

Both workflows run tests / CI only. No GitHub API writes, so workflow-level contents: read is the right cap.

Post-CVE-2025-30066 (tj-actions/changed-files) hardening pattern. YAML validated locally.

…quest

Both workflows run tests / CI only; no GitHub API writes.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@arpitjain099 arpitjain099 requested a review from a team as a code owner May 25, 2026 22:50
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented May 25, 2026

This pull request does not have a backport label. Could you fix it @arpitjain099? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@arpitjain099
Copy link
Copy Markdown
Author

Hey, I don't have write on the repo so can't add a label myself. For a workflow-perms PR like this I'd suggest backport-skip since the change is on the CI workflow file (not application code) and release branches typically don't get CI hardening pushed back. If you'd rather backport to an active branch, happy to follow up; just point me at the right backport-v8.x.x label and I'll mark it in a comment so a maintainer can apply.

@arpitjain099
Copy link
Copy Markdown
Author

Closing as duplicate of #6311 (earlier PR with overlapping scope), apologies for the noise.

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