-
-
Notifications
You must be signed in to change notification settings - Fork 713
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
Alert help should link to the alert pages #6014
base: main
Are you sure you want to change the base?
Conversation
All contributors have signed the CLA ✍️ ✅ |
@@ -28,6 +28,8 @@ public default String getHelpLink() { | |||
} | |||
|
|||
default String getHelpLink(int id) { | |||
|
|||
Alert ID: <a href="https://www.zaproxy.org/docs/alerts/90020/">90020</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the wrong place for this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you let me know the correct place to address this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the help content.
If you're trying to work on: zaproxy/zaproxy#8189
You need to read through the history. The only thing left to tackle is the WebSocket Passive rules (scripts).
I have read the CLA Document and I hereby sign the CLA |
Proposed solution: The Alert help should link to the alert pages using a standard format. Suggestion is to include a link immediately after the alert heading