zaproxy / zaproxy/zaproxy

Firing Range: Fix all Remote Inclusion and Reflected XSS results

Open
#7,123 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

add-on FalseNegative
Dominant language
Java
Stars
15.8k
Forks
2.6k
Avg merge
13h 16m
Merged PRs (30d)
17

Description

We run ZAP against Google Firing Range (FR) using a scheduled task and publish the results on https://www.zaproxy.org/docs/scans/firingrange/

There are currently:

  • 1 Reflected XSS failure
  • 2 Remote Inclusion XSS failures

It is important to note that the priority here is to improve the ZAP scan results against real work apps so any improvements should be generic rather than tailored for FR.

For more information about improving scan rules see https://www.zaproxy.org/docs/contribute/scan-rules/
In this case the relevant scan rule is linked to from each test.
As always all PRs should include full unit tests.

It is possible that some of the FR tests are no longer valid due to browser security improvements. If you believe this to be the case then please let us know and we will do our best to confirm that.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the Google Firing Range scan results and the relevant scan-rule links associated with each failing test, using the scan-rules contribution guide as context. Determine whether the Reflected XSS and Remote Inclusion failures are valid, improve the rules generically rather than for Firing Range, and include full unit tests. Done means the failures are resolved or their invalidity is confirmed.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
security, testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.