VulnerableApp icon indicating copy to clipboard operation
VulnerableApp copied to clipboard

Add unit test for Error based SQL Injection

Open preetkaran20 opened this issue 3 years ago • 5 comments
trafficstars

Is your feature request related to a problem? Please describe. We are looking at increasing the Test coverage for the project for its stability so this task is to add junit tests for https://github.com/SasanLabs/VulnerableApp/blob/master/src/main/java/org/sasanlabs/service/vulnerability/sqlInjection/ErrorBasedSQLInjectionVulnerability.java

preetkaran20 avatar Oct 01 '22 13:10 preetkaran20

Parent issue: https://github.com/SasanLabs/VulnerableApp/issues/212

preetkaran20 avatar Oct 01 '22 13:10 preetkaran20

Hi, is this issue still available? I would like to work on it

13Anthony avatar Oct 07 '23 17:10 13Anthony

Hi @13Anthony,

Yes this issue is available. Assigning the ticket to you.

thanks, Karan

preetkaran20 avatar Oct 08 '23 21:10 preetkaran20

Hi, I've added tests, but I'm not able to push my changes. I got this error: remote: Permission to SasanLabs/VulnerableApp.git denied to 13Anthony. fatal: unable to access 'https://github.com/SasanLabs/VulnerableApp.git/': The requested URL returned error: 403 I generated token in github with all permissions and created my own branch too. Do you have any idea where the problem can be?

Thank you in advance.

13Anthony avatar Nov 08 '23 10:11 13Anthony

@13Anthony you need to create a fork and from fork you need to create the pull request. Direct PR creation via branches is not allowed.

preetkaran20 avatar Nov 08 '23 18:11 preetkaran20