SSTImap icon indicating copy to clipboard operation
SSTImap copied to clipboard

Parameter support issues.

Open sectroyer opened this issue 2 years ago • 8 comments

SSTImap lacks '-p' (or equivalent) switch for specifying injection parameter.

Also it would be nice to have some improvement in case of multiple parameters like here:

[*] Javascript plugin is testing rendering with tag '*'
[*] Javascript plugin is testing ;*// code context escape with 6 variations
[*] Javascript plugin is testing blind injection
[*] Javascript plugin is testing ;*// code context escape with 6 variations
[*] Testing if POST parameter 'csrf' is injectable
[*] Ejs plugin is testing rendering with tag '*'
[*] Ejs plugin is testing %>*<%# code context escape with 6 variations
[*] Ejs plugin is testing blind injection
[*] Ejs plugin is testing %>*<%# code context escape with 6 variations
[*] Freemarker plugin is testing rendering with tag '*'
[*] Freemarker plugin is testing }* code context escape with 6 va

It's hard to find a line where it switches to new parameter. Even change to something like this would help a lot:

[*] Javascript plugin is testing rendering with tag '*'
[*] Javascript plugin is testing ;*// code context escape with 6 variations
[*] Javascript plugin is testing blind injection
[*] Javascript plugin is testing ;*// code context escape with 6 variations

[*] Testing if POST parameter 'csrf' is injectable
[*] Ejs plugin is testing rendering with tag '*'
[*] Ejs plugin is testing %>*<%# code context escape with 6 variations
[*] Ejs plugin is testing blind injection
[*] Ejs plugin is testing %>*<%# code context escape with 6 variations
[*] Freemarker plugin is testing rendering with tag '*'
[*] Freemarker plugin is testing }* code context escape with 6 va

Maybe even a different color of the "Testing..." line...

sectroyer avatar May 18 '23 15:05 sectroyer

I will probably change the color. Also, have you set an injection marker (*) as a parameter you need?

vladko312 avatar May 20 '23 06:05 vladko312

Nope I didn't. It wasn't clear for me if it works or not :)

sectroyer avatar May 25 '23 15:05 sectroyer

I made URL/form and parameter changing stand out a bit more.

Can you verify?

As for marker usage, it requires some documentation, so the issue will remain open for now.

vladko312 avatar May 26 '23 19:05 vladko312

Yes this green color looks much better 👍

sectroyer avatar Jun 01 '23 13:06 sectroyer

Thank you for your feedback! I will close this issue after creating some documentation.

vladko312 avatar Jun 01 '23 13:06 vladko312

After some more testing I have noticed one issue with current logging: [*] Testing if POST parameter 'TEST' is injectable Since it's in green and ends with "parameter XYZ is injectable" it often confuses me. Tough I know the tool and notice it after a second. Still I think something like this: [*] Testing injection on POST parameter 'TEST' Especially since sqlmap logs "... is injectable" so that's probably why it confuses me 😄

sectroyer avatar Jun 14 '23 16:06 sectroyer

Maybe, I will change the colour to yellow and change the text as well

vladko312 avatar Jun 14 '23 17:06 vladko312

Should be more clear in 1.2.0 Can you verify?

vladko312 avatar Jan 06 '24 23:01 vladko312