Skip to content

Commit b0e50b0

Browse files
leliaclaude
andcommitted
fix(dependency-review): require strict org membership for enterprise SFW
Tighten the enterprise-mode gate to author_association OWNER/MEMBER only. Outside collaborators (COLLABORATOR) now fall through to the free edition, same as Dependabot / forks / external contributors. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 37d06ad commit b0e50b0

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

.github/workflows/dependency-review.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -83,13 +83,14 @@ jobs:
8383
SOCKET_API_TOKEN: ${{ secrets.SOCKET_API_TOKEN }}
8484
run: |
8585
mode=firewall-free
86-
# Enterprise only for a trusted SocketDev member (OWNER/MEMBER) or
87-
# repo collaborator on an in-repo PR, and only when the token is
88-
# actually present. Anything else falls back to the free edition.
86+
# Enterprise only for a SocketDev org member (OWNER/MEMBER) on an
87+
# in-repo PR, and only when the token is actually present. Everything
88+
# else -- Dependabot, forks, outside collaborators, external
89+
# contributors, or a missing token -- uses the free edition.
8990
if [ "$IS_DEPENDABOT" != "true" ] \
9091
&& [ "$IS_FORK" != "true" ] \
9192
&& [ -n "$SOCKET_API_TOKEN" ] \
92-
&& printf '%s' "$AUTHOR_ASSOC" | grep -qE '^(OWNER|MEMBER|COLLABORATOR)$'; then
93+
&& printf '%s' "$AUTHOR_ASSOC" | grep -qE '^(OWNER|MEMBER)$'; then
9394
mode=firewall-enterprise
9495
fi
9596

0 commit comments

Comments
 (0)