Skip to content

Commit 25dc09c

Browse files
committed
fix: add allow-list so people can't override to random stuff
1 parent e935445 commit 25dc09c

1 file changed

Lines changed: 23 additions & 1 deletion

File tree

.github/workflows/dependency-review-vulnerability.yml

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,29 @@ jobs:
2929
fetch-depth: 0
3030
persist-credentials: false
3131

32+
- name: Choose config preset
33+
id: preset
34+
env:
35+
DEFAULT_PRESET: 'vulnerability-high'
36+
OVERRIDE_PRESET: ${{ vars.DEPENDENCY_REVIEW_VULNERABILITY_CONFIG_PRESET }}
37+
ALLOWED_PRESETS: 'vulnerability-high vulnerability-high-cve-2026-34040'
38+
run: |
39+
if [ -z "${OVERRIDE_PRESET:-}" ]; then
40+
echo "Using default config preset: $DEFAULT_PRESET"
41+
CONFIG_PRESET="$DEFAULT_PRESET"
42+
else
43+
if echo " $ALLOWED_PRESETS " | grep -q " $OVERRIDE_PRESET "; then
44+
echo "Using override config preset: $OVERRIDE_PRESET"
45+
CONFIG_PRESET="$OVERRIDE_PRESET"
46+
else
47+
echo "::warning::Invalid override preset '$OVERRIDE_PRESET'. Falling back to default '$DEFAULT_PRESET'."
48+
CONFIG_PRESET="$DEFAULT_PRESET"
49+
fi
50+
fi
51+
52+
echo "preset=$CONFIG_PRESET" | tee -a "$GITHUB_OUTPUT"
53+
3254
- name: Dependency Review
3355
uses: smartcontractkit/.github/actions/dependency-review@dependency-review/v2
3456
with:
35-
config-preset: ${{ vars.DEPENDENCY_REVIEW_VULNERABILITY_CONFIG_PRESET || 'vulnerability-high' }}
57+
config-preset: ${{ steps.preset.outputs.preset }}

0 commit comments

Comments
 (0)