Skip to content

Remove zap baseline and zap advanced scanner#3081

Merged
J12934 merged 8 commits intosecureCodeBox:mainfrom
Reet00:remove-zap-baseline-and-advanced-scanner
Jun 14, 2025
Merged

Remove zap baseline and zap advanced scanner#3081
J12934 merged 8 commits intosecureCodeBox:mainfrom
Reet00:remove-zap-baseline-and-advanced-scanner

Conversation

@Reet00
Copy link
Contributor

@Reet00 Reet00 commented Jun 13, 2025

Description

Remove the ZAP-baseline and ZAP-advanced scanner in favor of the zap-automation-framework.

  • Removed scanner directories
  • Replaced the removed scanner in the documentation

closes #3057

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

Reet00 added 8 commits June 13, 2025 22:31
in favor of the ZAP-automation-framework

- Deleted scanner directories
- Removed scanner from CI, release-build and scb-bot

Signed-off-by: Samreet Singh <[email protected]>
(zap-baseline-scan and zap-advanced-scan)

- Changed examples with zap files with either nmap or zap-automation-framework
- removed section "Reusing components from other scanners" in Makefile.md
because this was only applicable to zap-advanced

Signed-off-by: Samreet Singh <[email protected]>
The provided parameters were the same as for the zap-advanced and
therefore not compatible with the zap-automation-framework.

Signed-off-by: Samreet Singh <[email protected]>
and replace ZAP-script with simpler solution

Signed-off-by: Samreet Singh <[email protected]>
Signed-off-by: Samreet Singh <[email protected]>
@Reet00 Reet00 added scanner Implement or update a security scanner breaking Changes requiring a major release labels Jun 13, 2025
@netlify
Copy link

netlify bot commented Jun 13, 2025

Deploy Preview for docs-securecodebox ready!

Name Link
🔨 Latest commit eaf23bb
🔍 Latest deploy log https://app.netlify.com/projects/docs-securecodebox/deploys/684c8c459343fe0008c53013
😎 Deploy Preview https://deploy-preview-3081--docs-securecodebox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Reet00 Reet00 self-assigned this Jun 13, 2025
@secureCodeBoxBot
Copy link
Contributor

This pull request includes breaking changes. Please make sure that you included the breaking changes and the steps required to upgrade in Upgrading.
✨ Thank you for your contribution! ✨

@sonarqubecloud
Copy link

@github-project-automation github-project-automation bot moved this from Triage to Reviewer Approved in secureCodeBox Jun 14, 2025
@J12934 J12934 merged commit f38f276 into secureCodeBox:main Jun 14, 2025
51 checks passed
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in secureCodeBox Jun 14, 2025
@Reet00 Reet00 mentioned this pull request Jun 16, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Changes requiring a major release scanner Implement or update a security scanner

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Remove ZAP Baselines

3 participants

Comments