Skip to content

Add explicit read-only permissions to reusable workflows#3243

Open
arpitjain099 wants to merge 1 commit into
apache:masterfrom
arpitjain099:security/workflow-permissions-reusable-workflows
Open

Add explicit read-only permissions to reusable workflows#3243
arpitjain099 wants to merge 1 commit into
apache:masterfrom
arpitjain099:security/workflow-permissions-reusable-workflows

Conversation

@arpitjain099
Copy link
Copy Markdown

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.

@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.79%. Comparing base (9fbb61d) to head (acbabd2).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3243      +/-   ##
============================================
+ Coverage     73.77%   73.79%   +0.02%     
  Complexity      943      943              
============================================
  Files          1190     1190              
  Lines        107709   107832     +123     
  Branches      84726    84867     +141     
============================================
+ Hits          79461    79579     +118     
+ Misses        25497    25473      -24     
- Partials       2751     2780      +29     
Components Coverage Δ
Rust Core 74.86% <ø> (+0.03%) ⬆️
Java SDK 60.14% <ø> (ø)
C# SDK 69.13% <ø> (-0.31%) ⬇️
Python SDK 81.43% <ø> (ø)
Node SDK 91.53% <ø> (+0.12%) ⬆️
Go SDK 39.80% <ø> (ø)
see 38 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.

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