Skip to content

Comments

fix: resolve ZAP baseline scan report filename mismatch#3187

Closed
HarshitVerma109 wants to merge 1 commit intoOWASP:mainfrom
HarshitVerma109:fix/zap-baseline-scan-report-filename
Closed

fix: resolve ZAP baseline scan report filename mismatch#3187
HarshitVerma109 wants to merge 1 commit intoOWASP:mainfrom
HarshitVerma109:fix/zap-baseline-scan-report-filename

Conversation

@HarshitVerma109
Copy link
Contributor

@HarshitVerma109 HarshitVerma109 commented Jan 5, 2026

Proposed change

Resolves #3183

This PR fixes the CI/CD pipeline failure in the ZAP Baseline Scan jobs. The zaproxy/action-baseline action expects the HTML report to be named report_html.html by default, but the workflow configuration was overriding this with -r zap-report.html in cmd_options, causing a file not found error.

Changes made:

  • Remove -r zap-report.html from cmd_options to use default filename
  • Update artifact upload path to report_html.html
  • Fixes both staging and production scan jobs
  • Resolves issue where action expected report_html.html but got zap-report.html

Files modified:

  • .github/workflows/run-ci-cd.yaml

Checklist

  • Required: I read and followed the contributing guidelines
  • Required: I ran make check-test locally and all tests passed
  • I used AI for code, documentation, or tests in this PR

   - Remove -r zap-report.html from cmd_options to use default filename
   - Update artifact upload path to report_html.html
   - Fixes both staging and production scan jobs
   - Resolves issue where action expected report_html.html but got zap-report.html
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 5, 2026

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated CI/CD workflow configuration for improved clarity and maintainability.
    • Enhanced job display names in GitHub Actions for better readability.
    • Refined security scanning tool configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

CI/CD workflow configuration updates to the GitHub Actions pipeline. Job display names changed to capitalized forms in staging and production sections. ZAP baseline scan command options simplified from '-a -r zap-report.html' to '-a', and artifact upload path changed from zap-report.html to report_html.html.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/run-ci-cd.yaml
Updated job display names for staging and production image builds to use capitalized titles. Modified ZAP baseline scan configuration: simplified command options (removed report generation flag) and changed artifact upload path from zap-report.html to report_html.html in both staging and production workflow sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

ci

Suggested reviewers

  • arkid15r
  • kasya
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 572877b and c864879.

📒 Files selected for processing (1)
  • .github/workflows/run-ci-cd.yaml

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 5, 2026

PR validation failed: No linked issue and no valid closing issue reference in PR description

@github-actions github-actions bot closed this Jan 5, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 5, 2026

@HarshitVerma109 HarshitVerma109 deleted the fix/zap-baseline-scan-report-filename branch January 5, 2026 02:02
@HarshitVerma109 HarshitVerma109 restored the fix/zap-baseline-scan-report-filename branch January 5, 2026 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix ZAP baseline scan issues

1 participant