Skip to content

Commit 2df7c61

Browse files
authored
Merge pull request sherlock-project#2939 from sherlock-project/fix-vuln
Fix command injection vuln
2 parents 271608f + 61aae78 commit 2df7c61

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

.github/workflows/validate_modified_targets.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ jobs:
2020
# Checkout the base branch but fetch all history to avoid a second fetch call
2121
ref: ${{ github.base_ref }}
2222
fetch-depth: 0
23+
persist-credentials: false
2324

2425
- name: Set up Python
2526
uses: actions/setup-python@v6
@@ -90,11 +91,11 @@ jobs:
9091
# --- The rest of the steps below are unchanged ---
9192

9293
- name: Validate modified targets
93-
if: steps.discover-modified.outputs.changed_targets != ''
94-
continue-on-error: true
94+
env:
95+
CHANGED_TARGETS: ${{ steps.discover-modified.outputs.changed_targets }}
9596
run: |
9697
poetry run pytest -q --tb no -rA -m validate_targets -n 20 \
97-
--chunked-sites "${{ steps.discover-modified.outputs.changed_targets }}" \
98+
--chunked-sites "$CHANGED_TARGETS" \
9899
--junitxml=validation_results.xml
99100
100101
- name: Prepare validation summary

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ source = "init"
88

99
[tool.poetry]
1010
name = "sherlock-project"
11-
version = "0.16.0"
11+
version = "0.16.1"
1212
description = "Hunt down social media accounts by username across social networks"
1313
license = "MIT"
1414
authors = [

0 commit comments

Comments
 (0)