Skip to content

Commit

Permalink
feat(release): 4.15.0 (#218)
Browse files Browse the repository at this point in the history
Release notes:

- refactor: semgrep job update #213
#213
- feat: Remove virustotal
#217
- fix: ADDON-67588 internal repo installation
#215

---------

Co-authored-by: kdoroszko-splunk <[email protected]>
Co-authored-by: kdoroszko-splunk <[email protected]>
Co-authored-by: Marcin Bruzda <[email protected]>
  • Loading branch information
4 people authored Jan 16, 2024
1 parent 7e6a5a9 commit 44070f0
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 47 deletions.
46 changes: 16 additions & 30 deletions .github/workflows/reusable-build-test-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@ on:
AWS_SECRET_ACCESS_KEY:
description: AWS secret access key
required: true
VT_API_KEY:
description: Virustotal api key
required: true
OTHER_TA_REQUIRED_CONFIGS:
description: other required configs
required: true
Expand Down Expand Up @@ -369,14 +366,16 @@ jobs:
name: security-sast-semgrep
needs:
- setup-workflow
if: ${{ needs.setup-workflow.outputs.skip-workflow != 'Yes' && github.actor != 'dependabot[bot]' }}
container:
image: returntocorp/semgrep
if: ${{ needs.setup-workflow.outputs.skip-workflow != 'Yes' }}
steps:
- uses: actions/checkout@v3
- name: Semgrep
id: semgrep
uses: semgrep/semgrep-action@v1
with:
publishToken: ${{ secrets.SEMGREP_PUBLISH_TOKEN }}
run: semgrep ci
env:
SEMGREP_APP_TOKEN: ${{ secrets.SEMGREP_PUBLISH_TOKEN }}

test-inventory:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -441,6 +440,7 @@ jobs:
poetry export --without-hashes --dev -o requirements_dev.txt
fi
if [ ! -f requirements_dev.txt ]; then echo no requirements;exit 0 ;fi
git config --global url."https://${{ secrets.GH_TOKEN_ADMIN }}@github.com".insteadOf https://github.com
pip install -r requirements_dev.txt
- name: Create directories
run: |
Expand Down Expand Up @@ -489,6 +489,7 @@ jobs:
poetry export --without-hashes --dev -o requirements_dev.txt
fi
if [ ! -f requirements_dev.txt ]; then echo no requirements;exit 0 ;fi
git config --global url."https://${{ secrets.GH_TOKEN_ADMIN }}@github.com".insteadOf https://github.com
pip install -r requirements_dev.txt
- name: Create directories
run: |
Expand Down Expand Up @@ -569,7 +570,9 @@ jobs:
${{ runner.os }}-pip-
- name: Install deps
if: ${{ steps.checklibs.outputs.ENABLED == 'true' }}
run: pip install -r requirements_dev.txt
run: |
git config --global url."https://${{ secrets.GH_TOKEN_ADMIN }}@github.com".insteadOf https://github.com
pip install -r requirements_dev.txt
- name: Semantic Release Get Next
id: semantic
if: github.event_name != 'pull_request'
Expand Down Expand Up @@ -699,7 +702,9 @@ jobs:
key: ${{ runner.os }}-pip-python3_9-${{ hashFiles('requirements_dev.txt') }}
restore-keys: |
${{ runner.os }}-pip-python3_9
- run: pip install -r requirements_dev.txt
- run: |
git config --global url."https://${{ secrets.GH_TOKEN_ADMIN }}@github.com".insteadOf https://github.com
pip install -r requirements_dev.txt
- id: semantic
if: github.event_name != 'pull_request'
uses: splunk/[email protected]
Expand All @@ -721,24 +726,6 @@ jobs:
with:
version: ${{ steps.BuildVersion.outputs.VERSION }}

security-virustotal:
continue-on-error: true
name: security-virustotal
needs: build
if: ${{ !cancelled() && needs.build.result == 'success' }}
runs-on: ubuntu-latest
steps:
- uses: actions/download-artifact@v3
with:
name: package-splunkbase
path: build/package/
- name: VirusTotal Scan
uses: crazy-max/ghaction-virustotal@v4
with:
vt_api_key: ${{ secrets.VT_API_KEY }}
files: |
build/package/*
run-requirements-unit-tests:
runs-on: ubuntu-latest
needs:
Expand Down Expand Up @@ -848,9 +835,9 @@ jobs:
artifact-registry:
runs-on: ubuntu-latest
needs:
- security-virustotal
- meta
if: ${{ !cancelled() && needs.security-virustotal.result == 'success' && needs.meta.result == 'success' }}
- build
if: ${{ !cancelled() && needs.build.result == 'success' && needs.meta.result == 'success' }}
outputs:
artifact: ${{ steps.artifactid.outputs.result }}
permissions:
Expand Down Expand Up @@ -2682,7 +2669,6 @@ jobs:
- review_secrets
- semgrep
- build
- security-virustotal
- test-inventory
- run-unit-tests
- appinspect
Expand Down
20 changes: 3 additions & 17 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -213,9 +213,9 @@ security-sast-semgrep
**Description:**

- Semgrep CI behaves like other static analysis and linting tools: it runs a set of user-configured rules and returns a non-zero exit code if there are findings, resulting in its job showing a ✅ or ❌.

**Action used:** https://github.com/returntocorp/semgrep-action

- Semgrep can do two scan types:
- diff-aware scan, performed while workflow triggering event is pull request and scans only changes in files, which keeps the scan fast and reduces finding duplication.
- full scan, performed while workflow triggering event is other event (e.g. push) and scans the whole codebase.

**Pass/fail behaviour**

Expand Down Expand Up @@ -299,20 +299,6 @@ installation-update.json
```
- package-splunkbase includes Splunkbase equivalent package code

security-virustotal
=======================

**Description**

GitHub Action to upload and scan files with VirusTotal which analyze files, domains, IPs and URLs to detect malware, suspicions and other breaches

**Action used** https://github.com/crazy-max/ghaction-virustotal

**Artifacts:**

- No Artifacts for this stage. Report link is available in the logs
- <img src="images/virus-total/stage.png" alt="stage" style="width:200px;"/> <img src="images/virus-total/platform.png" alt="platform" style="width:200px;"/>


AppInspect
=======================
Expand Down
Binary file removed images/virus-total/platform.png
Binary file not shown.
Binary file removed images/virus-total/stage.png
Binary file not shown.

0 comments on commit 44070f0

Please sign in to comment.