Skip to content

Commit b231b75

Browse files
authored
fix zizmor findings (#3200)
* fix zizmor findings * fix bad persist-credentials
1 parent ee31969 commit b231b75

3 files changed

Lines changed: 17 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ name: Continuous Integration
22

33
on: pull_request
44

5+
permissions:
6+
contents: read
7+
58
jobs:
69
build:
710

@@ -13,6 +16,8 @@ jobs:
1316

1417
steps:
1518
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
19+
with:
20+
persist-credentials: false
1621
- name: Set up Node.js ${{ matrix.node-version }}
1722
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
1823
with:

.github/workflows/nightly.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ on:
2222
repository_dispatch:
2323
types: [firebase_nightly_build]
2424

25+
permissions:
26+
contents: read
27+
2528
jobs:
2629
nightly:
2730

@@ -32,6 +35,7 @@ jobs:
3235
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
3336
with:
3437
ref: ${{ github.event.client_payload.ref || github.ref }}
38+
persist-credentials: false
3539

3640
- name: Set up Node.js
3741
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0

.github/workflows/release.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ on:
3232
types:
3333
- firebase_build
3434

35+
permissions:
36+
contents: read
37+
3538
jobs:
3639
stage_release:
3740
# To publish a release, merge a PR with the title prefix '[chore] Release ' to main
@@ -47,6 +50,8 @@ jobs:
4750
steps:
4851
- name: Checkout source for staging
4952
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
53+
with:
54+
persist-credentials: false
5055

5156
- name: Set up Node.js
5257
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
@@ -110,6 +115,9 @@ jobs:
110115
steps:
111116
- name: Checkout source for publish
112117
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
118+
with:
119+
# Ignored since the GITHUB_TOKEN needs to be persisted for this workflow.
120+
persist-credentials: true # zizmor: ignore[artipacked]
113121

114122
# Download the artifacts created by the stage_release job.
115123
- name: Download release candidates

0 commit comments

Comments
 (0)