Repository navigation
Add Referrer-Policy header check and update description - #518
balaakasam wants to merge 3 commits into
Conversation
Signed-off-by: balaakasam <thripuraavula@gmail.com>
|
New Issues (2 out of 2)Checkmarx found the following issues in this Pull Request
Fixed Issues (1)Great job! The following issues were fixed in this Pull Request
Use @Checkmarx to take action directly from this PR:
Try it: |
|
Needs a spotlessApply. |
Signed-off-by: balaakasam <thripuraavula@gmail.com>
|
Thank you for the feedback @kingthorin. I've refactored the Referrer-Policy check to use the existing array mechanism consistent with the rest of the script. Please let me know if any further changes are needed, I can do so accordingly. |
|
@kingthorin ,wondering if this need PR any more changes from me? or is it ready to be published? |
Signed-off-by: Rick M <kingthorin@users.noreply.github.com>
|
@ balaakasam the PR needs the spotlessApply task run. Are you able to do that? |
|
The CHANCELOG should also be updated. |


Description
Adds a missing
Referrer-Policyheader check to the existing Multiple Security Header Check script and updates the metadata description to include it in the list of checked headers.Changes
Referrer-Policyto the metadata description listReferrer-Policyheader with appropriate alert name, description, and solutionWhy
The Referrer-Policy header is a modern security header recommended by OWASP and included in the OWASP Secure Headers Project. It was missing from this script while all other major security headers were already covered.
References