Skip to content

ci: scope down GitHub Token permissions#2762

Merged
justsmth merged 31 commits intoaws:mainfrom
AdnaneKhan:update/scopedown-token
Oct 22, 2025
Merged

ci: scope down GitHub Token permissions#2762
justsmth merged 31 commits intoaws:mainfrom
AdnaneKhan:update/scopedown-token

Conversation

@AdnaneKhan
Copy link
Copy Markdown
Contributor

@AdnaneKhan AdnaneKhan commented Oct 21, 2025

Scope Down GitHub Token Permissions

This PR updates GitHub Actions workflows to use minimal required permissions instead of the default elevated permissions.

Why This Matters

Following the principle of least privilege, workflows should only have the specific permissions they need to function.

Changes

This PR adds explicit permissions: blocks to workflows that currently rely on default permissions, scoping them down to only what's required for their operations.

Please review the changes to ensure the specified permissions match your workflow requirements.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@AdnaneKhan AdnaneKhan marked this pull request as ready for review October 21, 2025 20:58
@AdnaneKhan AdnaneKhan requested a review from a team as a code owner October 21, 2025 20:58
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.65%. Comparing base (e1eddc7) to head (512763b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2762      +/-   ##
==========================================
- Coverage   78.65%   78.65%   -0.01%     
==========================================
  Files         678      678              
  Lines      116334   116334              
  Branches    16314    16314              
==========================================
- Hits        91500    91499       -1     
- Misses      24045    24047       +2     
+ Partials      789      788       -1     

☔ View full report in Codecov by Sentry.
📢 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.

@justsmth justsmth enabled auto-merge (squash) October 21, 2025 23:01
@justsmth justsmth disabled auto-merge October 22, 2025 17:16
@justsmth justsmth merged commit 16c148a into aws:main Oct 22, 2025
358 of 377 checks passed
samuel40791765 pushed a commit to samuel40791765/aws-lc that referenced this pull request Oct 23, 2025
This PR updates GitHub Actions workflows to use minimal required
permissions instead of the default elevated permissions.

Following the principle of least privilege, workflows should only have
the specific permissions they need to function.

This PR adds explicit `permissions:` blocks to workflows that currently
rely on default permissions, scoping them down to only what's required
for their operations.

Please review the changes to ensure the specified permissions match your
workflow requirements.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and the ISC license.
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.

4 participants