ilmila / J2EEScan

J2EEScan is a plugin for Burp Suite Proxy. The goal of this plugin is to improve the test coverage during web application penetration tests on J2EE applications.
GNU General Public License v2.0
645 stars 186 forks source link

UI not combining multiple findings #36

Open AkikoOrenji opened 2 years ago

AkikoOrenji commented 2 years ago

I noticed a few findings on my assessment were 'missing' and working with portswigger we narrowed it down to J2EEScan finding similar issues in different injection points in the same application. It was confirmed with Logger++ that J2EEScan doesn't roll up the finding in a similar manner to other extensions.

In the the attached screen shot you can see that NoSQL Injection Detected (from a different extension) rolls up mutiliple findings in different requests and injection points.

image

Even though this project has multiple findings for the same XXE in different locations i only see one finding. This makes it hard to validate the other findings (apart from the first ) as the Request and Response aren't logged anywhere (unless you are using additional logging which you need to go search through the find the other effected injection points). This issue also presents itself as an inconsistency throughout the UI (especially when using mutiple Audit tasks) as additional findings are shown in some areas but not others e.g.

e.g. Details page of audit screen shows 0 high severity issues:

image

Audit Items page shows 3 high severity issues (i confirmed these were J2EEScan issues)

image

Issue activity page shows no issue:

image

Issue activity summary page shows only one High severity issue for a different task.

image

Is this just me or is this a possible improvement that could be made to how mutliple issues are combined in J2EEScan

Let me know if you need any other info

ilmila commented 2 years ago

Thank you for your ticket. I ask you:

AkikoOrenji commented 2 years ago

Thanks for looking into this. With show all set as the filter its the same behaviour

image

Burp is v2021.12.1

Errors attached (anywhere you see 'redacted' the values were exactly the same as the others finding) j2ee-err.txt j2ee-output.txt