Skip to content

Ensure artifact path for DAST reports not is overwritten - #3026

Merged
sydseter merged 4 commits into
masterfrom
ensure-the-artifact-path
May 23, 2026
Merged

sydseter merged 4 commits into
masterfrom
ensure-the-artifact-path

Conversation

@sydseter

@sydseter sydseter commented May 23, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Ensure artifact path for DAST reports not is overwritten

AI Tool Disclosure

  • My contribution does not include any AI-generated content
  • My contribution includes AI-generated content, as disclosed below:
    • AI Tools: [e.g. GitHub CoPilot, ChatGPT, JetBrains Junie etc.]
    • LLMs and versions: [e.g. GPT-4.1, Claude Haiku 4.5, Gemini 2.5 Pro etc.]
    • Prompts: [Summarize the key prompts or instructions given to the AI tools]

Affirmation

Copilot AI review requested due to automatic review settings May 23, 2026 12:16
@sydseter
sydseter requested review from cw-owasp and rewtd as code owners May 23, 2026 12:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to prevent the DAST (OWASP ZAP) report artifact path for the website scan from being overwritten by switching the reports directory name used for uploads.

Changes:

  • Updated the upload-artifact path for website DAST reports to zap-website-reports/.
  • Updated downstream steps (vulnerability check file existence + release upload file list) to reference zap-website-reports/.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

@sydseter
sydseter merged commit e230220 into master May 23, 2026
18 of 19 checks passed
@sydseter
sydseter deleted the ensure-the-artifact-path branch May 23, 2026 18:04
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.

3 participants