Skip to content

ci: set read-only workflow token permissions - #1054

Open
Alb3e3 wants to merge 1 commit into
auth0:masterfrom
Alb3e3:alb3e3/ci-readonly-token-permissions
Open

ci: set read-only workflow token permissions#1054
Alb3e3 wants to merge 1 commit into
auth0:masterfrom
Alb3e3:alb3e3/ci-readonly-token-permissions

Conversation

@Alb3e3

@Alb3e3 Alb3e3 commented Jul 14, 2026

Copy link
Copy Markdown

Summary

  • set explicit read-only GITHUB_TOKEN permissions for CI workflows that only need repository checkout/read access
  • leave release/deploy/write-capable workflows unchanged

Why

This follows GitHub Actions least-privilege guidance and reduces the default token scope available to routine CI jobs without changing the test/build commands.

Verification

  • Parsed the changed workflow YAML with PyYAML.
  • Ran git diff --check.

@Alb3e3
Alb3e3 requested a review from a team as a code owner July 14, 2026 22:28
@kailash-b
kailash-b force-pushed the alb3e3/ci-readonly-token-permissions branch 2 times, most recently from 2f8d207 to e676b9d Compare July 17, 2026 07:42
@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.22%. Comparing base (dba5885) to head (e676b9d).
⚠️ Report is 27 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (dba5885) and HEAD (e676b9d). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (dba5885) HEAD (e676b9d)
authIntTests 1 0
mgmtIntTests 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1054       +/-   ##
===========================================
- Coverage   25.53%    0.22%   -25.32%     
===========================================
  Files        3156     3138       -18     
  Lines      148320   148192      -128     
  Branches    10021    10012        -9     
===========================================
- Hits        37874      330    -37544     
- Misses     108133   147853    +39720     
+ Partials     2313        9     -2304     
Flag Coverage Δ
authIntTests ?
mgmtIntTests ?
unittests 0.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Alb3e3
Alb3e3 force-pushed the alb3e3/ci-readonly-token-permissions branch from e676b9d to 1845ac9 Compare August 13, 2026 17:05
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