Skip to content

Commit bb07fba

Browse files
authored
fix zizmor findings (#16296)
1 parent 2032861 commit bb07fba

8 files changed

Lines changed: 15 additions & 7 deletions

File tree

.github/workflows/_build.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,11 @@ jobs:
154154
fi
155155
- name: Run setup command
156156
if: inputs.setup_command != ''
157-
run: ${{ inputs.setup_command }}
157+
# This file is used as a template for other workflows, so
158+
# using a template include here is necessary; however, this
159+
# means downstream consumers must be careful not to accept user
160+
# input into their setup_commands.
161+
run: ${{ inputs.setup_command }} # zizmor: ignore[template-injection]
158162
- name: Build
159163
uses: nick-fields/retry@ad984534de44a9489a53aefd81eb77f87c70dc60 # v4.0.0
160164
with:

.github/workflows/_cocoapods.cron.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ jobs:
9494
if: inputs.setup_command != ''
9595
env:
9696
plist_secret: ${{ secrets.plist_secret }}
97-
run: ${{ inputs.setup_command }}
97+
run: ${{ inputs.setup_command }} # zizmor: ignore[template-injection]
9898
- name: PodLibLint Cron
9999
uses: nick-fields/retry@ad984534de44a9489a53aefd81eb77f87c70dc60 # v4.0.0
100100
with:

.github/workflows/_cocoapods.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ jobs:
171171
if: inputs.setup_command != ''
172172
env:
173173
plist_secret: ${{ secrets.plist_secret }}
174-
run: ${{ inputs.setup_command }}
174+
run: ${{ inputs.setup_command }} # zizmor: ignore[template-injection]
175175
- name: Lint ${{ inputs.product }}.podspec for ${{ matrix.platform }}
176176
uses: nick-fields/retry@ad984534de44a9489a53aefd81eb77f87c70dc60 # v4.0.0
177177
if: contains(inputs.platforms, matrix.platform)

.github/workflows/_quickstart.framework.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ jobs:
9191
mkdir -p "${HOME}"/ios_frameworks/
9292
find "${GITHUB_WORKSPACE}" -name "Firebase*latest.zip" -exec unzip -d "${HOME}"/ios_frameworks/ {} +
9393
- name: Setup quickstart
94-
run: ${{ inputs.setup_command }}
94+
run: ${{ inputs.setup_command }} # zizmor: ignore[template-injection]
9595
- name: Install Secret GoogleService-Info.plist
9696
env:
9797
PLIST_SRC_PATH: ${{ inputs.plist_src_path }}

.github/workflows/_quickstart.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ jobs:
8888
continue_on_error: true
8989
command: xcodebuild -downloadPlatform iOS
9090
- name: Run setup command.
91-
run: ${{ inputs.setup_command }}
91+
run: ${{ inputs.setup_command }} # zizmor: ignore[template-injection]
9292
- name: Install Secret GoogleService-Info.plist
9393
env:
9494
PLIST_SRC_PATH: ${{ inputs.plist_src_path }}

.github/workflows/_spm.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ jobs:
171171
command: xcodebuild -downloadPlatform ${{ matrix.platform }}
172172
- name: Run setup command, if needed.
173173
if: inputs.setup_command != ''
174-
run: ${{ inputs.setup_command }}
174+
run: ${{ inputs.setup_command }} # zizmor: ignore[template-injection]
175175
- name: Initialize xcodebuild
176176
run: scripts/setup_spm_tests.sh
177177
- uses: nick-fields/retry@ad984534de44a9489a53aefd81eb77f87c70dc60 # v4.0.0

.github/workflows/infra.danger.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ jobs:
2626
with:
2727
app-id: ${{ secrets.DANGER_APP_ID }}
2828
private-key: ${{ secrets.DANGER_APP_PRIVATE_KEY }}
29+
permissions:
30+
contents: read
31+
issues: write
32+
pull-requests: write
2933

3034
- uses: ruby/setup-ruby@89f90524b88a01fe6e0b732220432cc6142926af # v1.313.0
3135
- name: Setup Bundler

.github/workflows/infra.ftl.nightly.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ jobs:
8888
- name: Xcode
8989
run: sudo xcode-select -s /Applications/Xcode_16.4.app/Contents/Developer
9090
- name: Setup quickstart
91-
run: ${{ matrix.setup_command }}
91+
run: ${{ matrix.setup_command }} # zizmor: ignore[template-injection]
9292
- name: Install Secret GoogleService-Info.plist
9393
run: |
9494
scripts/decrypt_gha_secret.sh \

0 commit comments

Comments
 (0)