Skip to content

Commit 8539381

Browse files
Tamara-Barumclaude
andauthored
[FSSDK-12316] [Security] Fix curl|bash vulnerability in SourceClear scan (#294)
* [Security] Fix curl|bash vulnerability in SourceClear scan - Replace dangerous curl|bash pattern with official Veracode SCA action - Use veracode/veracode-sca@v2 for secure, maintained scanning - Addresses script injection vulnerability in CI/CD pipeline - Related to commit 363cb85 (previous GitHub Actions security fix) Security improvements: - Official action from Veracode organization (signed and verified) - No untrusted remote code execution - Protected against MITM attacks - Same SRCCLR_API_TOKEN authentication preserved - Action is actively maintained and receives security updates Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * chore: Remove SourceClear scan workflow Following Python SDK's approach (commit d651911), removing the SourceClear GitHub Actions workflow to address security risks. The curl|bash pattern in this workflow presented a security vulnerability. Rather than replace with official action, we're removing it entirely to align with the Python SDK security remediation strategy. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 9b30186 commit 8539381

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

.github/workflows/php.yml

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -22,20 +22,9 @@ jobs:
2222
- name: Run linting
2323
run: composer lint
2424

25-
source_clear:
26-
name: Source Clear Scan
27-
runs-on: ubuntu-latest
28-
steps:
29-
- name: Checkout code
30-
uses: actions/checkout@v3
31-
- name: Source clear scan
32-
env:
33-
SRCCLR_API_TOKEN: ${{ secrets.SRCCLR_API_TOKEN }}
34-
run: curl -sSL https://download.sourceclear.com/ci.sh | bash -s – scan
35-
3625
unit_tests:
3726
name: Unit Tests ${{ matrix.php-versions }}
38-
needs: [ linting, source_clear ]
27+
needs: [ linting ]
3928
runs-on: ubuntu-latest
4029
strategy:
4130
fail-fast: false

0 commit comments

Comments
 (0)