Skip to content

fix: use gh release upload for ZAP report upload to pre-release#2339

Merged
sydseter merged 1 commit intoOWASP:masterfrom
Mysterio-17:fix/zap-prerelease-upload
Feb 21, 2026
Merged

fix: use gh release upload for ZAP report upload to pre-release#2339
sydseter merged 1 commit intoOWASP:masterfrom
Mysterio-17:fix/zap-prerelease-upload

Conversation

@Mysterio-17
Copy link
Copy Markdown
Contributor

Description

The softprops/action-gh-release action cannot upload files to an already existing release, so the nightly ZAP DAST reports were silently failing to attach to the pre-release. This replaces it with gh release upload --clobber, which properly uploads files to existing releases.

Closes #2333

Changes

  • Replaced softprops/action-gh-release with gh release upload "pre-release" --clobber
  • Each report file is checked for existence before upload (since ZAP scan uses || true)
  • Uses GITHUB_TOKEN for authentication (already available via permissions: contents: write)

@Mysterio-17
Copy link
Copy Markdown
Contributor Author

Hello @sydseter , I've replaced softprops/action-gh-release with gh release upload --clobber so ZAP reports are properly uploaded to the existing pre-release.
Kindly have a look at it.

@sydseter sydseter merged commit f3132ba into OWASP:master Feb 21, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Zap proxy is not uploading the reports to the pre-release

2 participants