Skip to content

ci: declare explicit read-only permissions on reusable workflows#3243

Merged
hubcio merged 4 commits into
apache:masterfrom
arpitjain099:security/workflow-permissions-reusable-workflows
May 14, 2026
Merged

ci: declare explicit read-only permissions on reusable workflows#3243
hubcio merged 4 commits into
apache:masterfrom
arpitjain099:security/workflow-permissions-reusable-workflows

Conversation

@arpitjain099
Copy link
Copy Markdown
Contributor

Summary

  • Add explicit permissions blocks with contents: read to reusable workflows:
    • .github/workflows/_detect.yml
    • .github/workflows/_build_rust_artifacts.yml
    • .github/workflows/_build_python_wheels.yml

Why

These reusable workflows perform checkout/build/packaging operations and do not require broad token privileges. Explicit read-only permissions tighten default GitHub token scope and make required access clear.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.81%. Comparing base (d158089) to head (584f171).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3243      +/-   ##
============================================
- Coverage     73.85%   73.81%   -0.05%     
  Complexity      943      943              
============================================
  Files          1193     1193              
  Lines        108933   108933              
  Branches      85950    85968      +18     
============================================
- Hits          80456    80406      -50     
- Misses        25742    25766      +24     
- Partials       2735     2761      +26     
Components Coverage Δ
Rust Core 74.87% <ø> (-0.03%) ⬇️
Java SDK 60.14% <ø> (ø)
C# SDK 69.13% <ø> (-0.31%) ⬇️
Python SDK 81.43% <ø> (ø)
Node SDK 91.41% <ø> (-0.13%) ⬇️
Go SDK 39.80% <ø> (ø)
see 23 files with indirect coverage changes
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hubcio
Copy link
Copy Markdown
Contributor

hubcio commented May 13, 2026

hi, thanks for contribution. please fix PR title.

@arpitjain099 arpitjain099 changed the title Add explicit read-only permissions to reusable workflows ci: declare explicit read-only permissions on reusable workflows May 13, 2026
@arpitjain099
Copy link
Copy Markdown
Contributor Author

Hi @hubcio — updated the PR title to follow the conventional-commits style used in this repo (ci: ...). Thanks for the nudge.

@arpitjain099 arpitjain099 force-pushed the security/workflow-permissions-reusable-workflows branch from 7428dfa to 2828b88 Compare May 13, 2026 17:13
@hubcio hubcio merged commit a5f85ea into apache:master May 14, 2026
81 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.

3 participants