Skip to content

Commit 183a94c

Browse files
committed
ci: replace secrets: inherit with explicit pass-throughs + suppress intentional artipacked
1 parent 9955969 commit 183a94c

12 files changed

Lines changed: 70 additions & 18 deletions

.github/workflows/changesets-pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
if: github.repository == 'triggerdotdev/trigger.dev'
2626
steps:
2727
- name: Checkout
28-
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
28+
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 # zizmor: ignore[artipacked] changesets/action pushes the release branch; no artifact upload here so no leak path
2929
with:
3030
fetch-depth: 0
3131

.github/workflows/e2e-webapp.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@ permissions:
55

66
on:
77
workflow_call:
8+
secrets:
9+
DOCKERHUB_USERNAME:
10+
required: false
11+
DOCKERHUB_TOKEN:
12+
required: false
813

914
jobs:
1015
e2eTests:

.github/workflows/pr_checks.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,18 +19,17 @@ permissions:
1919
jobs:
2020
typecheck:
2121
uses: ./.github/workflows/typecheck.yml
22-
secrets: inherit
2322

2423
units:
2524
uses: ./.github/workflows/unit-tests.yml
26-
secrets: inherit
25+
secrets:
26+
DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
27+
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
2728

2829
e2e:
2930
uses: ./.github/workflows/e2e.yml
3031
with:
3132
package: cli-v3
32-
secrets: inherit
3333

3434
sdk-compat:
3535
uses: ./.github/workflows/sdk-compat.yml
36-
secrets: inherit

.github/workflows/publish-webapp.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ on:
1313
type: string
1414
required: false
1515
default: ""
16+
secrets:
17+
SENTRY_AUTH_TOKEN:
18+
required: false
1619

1720
jobs:
1821
publish:

.github/workflows/publish-worker.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@ on:
88
type: string
99
required: false
1010
default: ""
11+
secrets:
12+
DOCKERHUB_USERNAME:
13+
required: false
14+
DOCKERHUB_TOKEN:
15+
required: false
1116
push:
1217
tags:
1318
- "infra-dev-*"

.github/workflows/publish.yml

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,13 @@ on:
88
description: The image tag to publish
99
required: true
1010
type: string
11+
secrets:
12+
DOCKERHUB_USERNAME:
13+
required: false
14+
DOCKERHUB_TOKEN:
15+
required: false
16+
SENTRY_AUTH_TOKEN:
17+
required: false
1118
push:
1219
branches:
1320
- main
@@ -48,11 +55,12 @@ env:
4855
jobs:
4956
typecheck:
5057
uses: ./.github/workflows/typecheck.yml
51-
secrets: inherit
5258

5359
units:
5460
uses: ./.github/workflows/unit-tests.yml
55-
secrets: inherit
61+
secrets:
62+
DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
63+
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
5664

5765
publish-webapp:
5866
needs: [typecheck]
@@ -61,7 +69,8 @@ jobs:
6169
packages: write
6270
id-token: write
6371
uses: ./.github/workflows/publish-webapp.yml
64-
secrets: inherit
72+
secrets:
73+
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
6574
with:
6675
image_tag: ${{ inputs.image_tag }}
6776

@@ -71,7 +80,9 @@ jobs:
7180
contents: read
7281
packages: write
7382
uses: ./.github/workflows/publish-worker.yml
74-
secrets: inherit
83+
secrets:
84+
DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
85+
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
7586
with:
7687
image_tag: ${{ inputs.image_tag }}
7788

@@ -82,6 +93,5 @@ jobs:
8293
packages: write
8394
id-token: write
8495
uses: ./.github/workflows/publish-worker-v4.yml
85-
secrets: inherit
8696
with:
8797
image_tag: ${{ inputs.image_tag }}

.github/workflows/release.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ jobs:
6666
published_package_version: ${{ steps.get_version.outputs.package_version }}
6767
steps:
6868
- name: Checkout repo
69-
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
69+
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 # zizmor: ignore[artipacked] needs persisted git creds for tag push; no artifact upload here so no leak path
7070
with:
7171
fetch-depth: 0
7272
ref: ${{ github.event_name == 'workflow_dispatch' && github.event.inputs.ref || github.sha }}
@@ -170,7 +170,10 @@ jobs:
170170
packages: write
171171
id-token: write
172172
uses: ./.github/workflows/publish.yml
173-
secrets: inherit
173+
secrets:
174+
DOCKERHUB_USERNAME: ${{ secrets.DOCKERHUB_USERNAME }}
175+
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
176+
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
174177
with:
175178
image_tag: v${{ needs.release.outputs.published_package_version }}
176179

@@ -185,7 +188,6 @@ jobs:
185188
contents: write
186189
packages: write
187190
uses: ./.github/workflows/release-helm.yml
188-
secrets: inherit
189191
with:
190192
chart_version: ${{ needs.release.outputs.published_package_version }}
191193

.github/workflows/unit-tests-internal.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@ permissions:
55

66
on:
77
workflow_call:
8+
secrets:
9+
DOCKERHUB_USERNAME:
10+
required: false
11+
DOCKERHUB_TOKEN:
12+
required: false
813

914
jobs:
1015
unitTests:

.github/workflows/unit-tests-packages.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@ permissions:
55

66
on:
77
workflow_call:
8+
secrets:
9+
DOCKERHUB_USERNAME:
10+
required: false
11+
DOCKERHUB_TOKEN:
12+
required: false
813

914
jobs:
1015
unitTests:

.github/workflows/unit-tests-webapp.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@ permissions:
55

66
on:
77
workflow_call:
8+
secrets:
9+
DOCKERHUB_USERNAME:
10+
required: false
11+
DOCKERHUB_TOKEN:
12+
required: false
813

914
jobs:
1015
unitTests:

0 commit comments

Comments
 (0)