Skip to content

#282 General page on dast tools. #283

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Sep 27, 2024
Merged

#282 General page on dast tools. #283

merged 10 commits into from
Sep 27, 2024

Conversation

sydseter
Copy link
Collaborator

Summary :

Fixes: #282

Description for the changelog :

Changing the ZAP page into a page about DAST in general

Other info :

More could be added, but this will give a good direction for now.

@sydseter sydseter requested a review from jgadsden September 27, 2024 08:45
@thc202
Copy link

thc202 commented Sep 27, 2024

Not opposing the change but would be good this change was not made based on FUD.

@sydseter
Copy link
Collaborator Author

true, but I think that is clarified now in #282
Perhaps something feels amiss?

@thc202
Copy link

thc202 commented Sep 27, 2024

Not with this PR, just that the issue title and description is still wrong/false.

@jgadsden jgadsden added the content suggest content label Sep 27, 2024
@jgadsden
Copy link
Collaborator

@thc202 the issue title is based on the new ZAP logo and the Checkmarx press release

Checkmarx is very clear about what is still free to the ZAP community and what will now be available only to Checkmarx customers

Screenshot 2024-09-27 at 14 32 09

@jgadsden
Copy link
Collaborator

@sydseter you are probably already editing these, but these files also need changes:

  • draft/08-verification/02-tools/00-toc.md
  • draft/08-verification/00-toc.md
  • draft/02-toc.md
  • _data/draft.yaml

@thc202
Copy link

thc202 commented Sep 27, 2024

#283 (comment)

If it's very clear, you should have no problems answering what features are becoming paid-for in ZAP (as the issue still falsely states: In addition some ZAP features will now become paid-for).

@jgadsden
Copy link
Collaborator

@sydseter the final error in the link checker will be corrected when the draft is promoted to release, so do not worry about trying to fix it

Copy link
Collaborator

@jgadsden jgadsden left a comment

Choose a reason for hiding this comment

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

thanks for doing this work (and accepting the challenge) @sydseter
everything looks like it is in place, so please go ahead and merge

@sydseter sydseter merged commit f479f95 into main Sep 27, 2024
@sydseter sydseter deleted the dast-tools branch September 27, 2024 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content suggest content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ZAP is now Checkmarx
4 participants