Skip to content

ci: declare workflow-level contents: read on 3 lint/check workflows#18956

Closed
arpitjain099 wants to merge 1 commit into
influxdata:masterfrom
arpitjain099:chore/declare-workflow-perms
Closed

ci: declare workflow-level contents: read on 3 lint/check workflows#18956
arpitjain099 wants to merge 1 commit into
influxdata:masterfrom
arpitjain099:chore/declare-workflow-perms

Conversation

@arpitjain099
Copy link
Copy Markdown
Contributor

Three workflows (pr-target-branch, readme-linter, semantic) just validate PR metadata or markdown. No GitHub API writes happen from the workflows.

Same hardening shape as the post-CVE-2025-30066 response (tj-actions/changed-files compromise). YAML validated locally with yaml.safe_load on each touched file.

Three lint/check workflows (pr-target-branch, readme-linter, semantic) just validate PR metadata or markdown. No GitHub API writes from the workflows.

Post-CVE-2025-30066 hardening shape (tj-actions/changed-files). yaml.safe_load validated on each touched file.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@telegraf-tiger telegraf-tiger Bot added the ci CI pipeline fixes, optimizations, and infrastructure label May 21, 2026
@srebhan
Copy link
Copy Markdown
Member

srebhan commented May 26, 2026

@arpitjain099 this is a duplicate of PR #18920, isn't it?

@srebhan srebhan added the waiting for response waiting for response from contributor label May 26, 2026
@srebhan srebhan self-assigned this May 26, 2026
@arpitjain099
Copy link
Copy Markdown
Contributor Author

@arpitjain099 this is a duplicate of PR #18920, isn't it?

@srebhan looks like it. I'll check more after I get home today

@telegraf-tiger telegraf-tiger Bot removed the waiting for response waiting for response from contributor label May 26, 2026
@arpitjain099
Copy link
Copy Markdown
Contributor Author

@srebhan yes you're right, exact duplicate of #18920 (which is already merged). Closing this out, apologies for the noise - my batch script missed that it had already landed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI pipeline fixes, optimizations, and infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants