Skip to content
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

pscanrulesAlpha: Link website alert pages <-> help #5094

Merged
merged 5 commits into from
Nov 16, 2023

Conversation

amitpanwar789
Copy link
Contributor

@amitpanwar789 amitpanwar789 commented Nov 15, 2023

Overview

added getHelpLink method in Base64Disclosure.java and FetchMetadataRequestHeadersScanRule.java
added anchor tag for respective method in pscanalpha.html

Related Issues

This PR is part of issue #8189
@psiinon can you please review it

Added

  • Website alert links (Issue 8189).

Checklist

  • Update help
  • Update changelog
  • Run ./gradlew spotlessApply for code formatting
  • Write tests
  • Check code coverage
  • Sign-off commits
  • Squash commits
  • Use a descriptive title

For more details, please refer to the developer rules and guidelines.

@thc202 thc202 changed the title Link website alert pages <-> help in Passive scan rules (alpha) . pscanrulesAlpha: Link website alert pages <-> help Nov 15, 2023
@psiinon
Copy link
Member

psiinon commented Nov 15, 2023

The changes look good.
You will need to sign the DCO (see the link above) and update the changelog e.g. with

### Added
- Website alert links (Issue 8189).

@psiinon
Copy link
Member

psiinon commented Nov 15, 2023

Oh, and to fix the tests:

Run './gradlew :addOns:pscanrulesAlpha:spotlessApply' to fix these violations.

Signed-off-by: amitpanwar789 <amitpanwar02705@gmail.com>
Signed-off-by: amitpanwar789 <amitpanwar02705@gmail.com>
Signed-off-by: amitpanwar789 <amitpanwar02705@gmail.com>
Signed-off-by: amitpanwar789 <amitpanwar02705@gmail.com>
Signed-off-by: thc202 <thc202@gmail.com>
@thc202 thc202 enabled auto-merge (squash) November 16, 2023 09:57
@thc202 thc202 merged commit 24c39d4 into zaproxy:main Nov 16, 2023
10 checks passed
@thc202
Copy link
Member

thc202 commented Nov 16, 2023

@amitpanwar789 thank you! How would you like to be credited (e.g. name, handle)?
https://www.zaproxy.org/docs/desktop/credits/#zap-extended-team

@thc202
Copy link
Member

thc202 commented Nov 28, 2023

@amitpanwar789 ping (it's fine if you don't want to, just making sure this was not missed).

@amitpanwar789
Copy link
Contributor Author

thanks for ping, (Amit Panwar, amitpanwar789) this would be good.

thc202 added a commit to thc202/zap-core-help that referenced this pull request Nov 28, 2023
From zaproxy/zap-extensions#5094.

Signed-off-by: thc202 <thc202@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants