Skip to content

Commit 1ca3aa7

Browse files
committed
Use CLI Socket token for enterprise dependency review
1 parent e653f3f commit 1ca3aa7

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

.github/workflows/dependency-review.yml

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,17 @@ name: dependency-review
77
# install smoke jobs for the affected manifests, picking the firewall edition
88
# per PR:
99
#
10-
# - Trusted SocketDev members on an in-repo (non-fork) PR, when the
11-
# SOCKET_API_TOKEN secret is present -> Socket Firewall ENTERPRISE
10+
# - Trusted SocketDev members on an in-repo (non-fork) PR, when
11+
# SOCKET_API_TOKEN or SOCKET_CLI_API_TOKEN is present -> Socket Firewall
12+
# ENTERPRISE
1213
# (authenticated, full org-policy enforcement).
1314
# - Everything else -- Dependabot, forks, external contributors, or a
1415
# missing token -> Socket Firewall FREE (anonymous, no API token), which
1516
# is safe in the unprivileged `pull_request` context.
1617
#
1718
# The mode degrades to free whenever the token is absent, so this workflow is
1819
# safe to ship before the secret exists and starts using enterprise
19-
# automatically once SOCKET_API_TOKEN is configured.
20+
# automatically once a Socket API token secret is configured.
2021
#
2122
# Pattern adapted from SocketDev/socket-basics.
2223

@@ -86,8 +87,8 @@ jobs:
8687
IS_DEPENDABOT: ${{ github.event.pull_request.user.login == 'dependabot[bot]' }}
8788
IS_FORK: ${{ github.event.pull_request.head.repo.full_name != github.repository }}
8889
AUTHOR_ASSOC: ${{ github.event.pull_request.author_association }}
89-
# Empty for fork PRs (secrets withheld) and until the secret is added.
90-
SOCKET_API_TOKEN: ${{ secrets.SOCKET_API_TOKEN }}
90+
# Empty for fork PRs (secrets withheld) and until a token secret is added.
91+
SOCKET_API_TOKEN: ${{ secrets.SOCKET_API_TOKEN || secrets.SOCKET_CLI_API_TOKEN }}
9192
run: |
9293
mode=firewall-free
9394
# Enterprise only for a trusted SocketDev member (OWNER/MEMBER) or
@@ -134,7 +135,7 @@ jobs:
134135
with:
135136
uv: "true"
136137
mode: ${{ needs.inspect.outputs.sfw_mode }}
137-
socket-token: ${{ secrets.SOCKET_API_TOKEN }}
138+
socket-token: ${{ secrets.SOCKET_API_TOKEN || secrets.SOCKET_CLI_API_TOKEN }}
138139

139140
- name: Sync project through Socket Firewall
140141
# `sfw uv sync` is the intended way to route uv through Socket Firewall
@@ -183,7 +184,7 @@ jobs:
183184
with:
184185
node: "true"
185186
mode: ${{ needs.inspect.outputs.sfw_mode }}
186-
socket-token: ${{ secrets.SOCKET_API_TOKEN }}
187+
socket-token: ${{ secrets.SOCKET_API_TOKEN || secrets.SOCKET_CLI_API_TOKEN }}
187188

188189
- name: Install fixture through Socket Firewall
189190
working-directory: tests/e2e/fixtures/simple-npm
@@ -204,7 +205,7 @@ jobs:
204205
with:
205206
python: "true"
206207
mode: ${{ needs.inspect.outputs.sfw_mode }}
207-
socket-token: ${{ secrets.SOCKET_API_TOKEN }}
208+
socket-token: ${{ secrets.SOCKET_API_TOKEN || secrets.SOCKET_CLI_API_TOKEN }}
208209

209210
- name: Install fixture through Socket Firewall
210211
working-directory: tests/e2e/fixtures/simple-pypi

0 commit comments

Comments
 (0)