Skip to content

Commit c5d0f0f

Browse files
Add explicit permissions to GitHub workflows (#2140)
Fix code scanning alert about unlimited permissions by applying the principle of least privilege to all workflow jobs. Each job now has only the permissions it actually needs: - Build, test, publish, and compat jobs get contents: read - publish-docs keeps contents: write (needs to push to gh-pages) - rules workflow gets empty permissions (no repo access needed) Committed-By-Agent: claude
1 parent 31c4551 commit c5d0f0f

2 files changed

Lines changed: 15 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,17 @@ on:
1818
- sdk-release/**
1919
- feature/**
2020

21+
permissions: {}
22+
2123
jobs:
2224
build:
2325
name: Build
2426

2527
runs-on: "ubuntu-24.04"
2628

29+
permissions:
30+
contents: read
31+
2732
steps:
2833
- uses: extractions/setup-just@v2
2934
- uses: actions/checkout@master
@@ -49,6 +54,9 @@ jobs:
4954

5055
runs-on: "ubuntu-24.04"
5156

57+
permissions:
58+
contents: read
59+
5260
strategy:
5361
fail-fast: false
5462
matrix:
@@ -106,6 +114,8 @@ jobs:
106114
endsWith(github.actor, '-stripe')
107115
needs: [build, test]
108116
runs-on: "ubuntu-24.04"
117+
permissions:
118+
contents: read
109119
steps:
110120
- uses: actions/checkout@master
111121
- name: Setup Java
@@ -169,6 +179,9 @@ jobs:
169179
compat:
170180
runs-on: "ubuntu-24.04"
171181

182+
permissions:
183+
contents: read
184+
172185
steps:
173186
- name: Checkout repository
174187
uses: actions/checkout@v2

.github/workflows/rules.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ on:
77
types:
88
- auto_merge_enabled
99

10+
permissions: {}
11+
1012
jobs:
1113
require_merge_commit_on_merge_script_pr:
1214
name: Merge script PRs must create merge commits

0 commit comments

Comments
 (0)