-
-
Notifications
You must be signed in to change notification settings - Fork 427
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SSRF and DoS vulnerabilities #365
Add SSRF and DoS vulnerabilities #365
Conversation
Hi @gled02 , |
1 similar comment
Hi @gled02 , |
Hi @preetkaran20 , I just raised the PR against ssrf_unmerged branch. Thanks |
Thanks !!! |
Hi @NMV01 , @gled02 , Regards |
Hi @preetkaran20, Thanks a lot for your comments. We are really happy to have been able to contribute to VulnerableApp and to help the improvement of this useful project. We are glad to have chosen VulnerableApp as an open source project, for our university project. We learned a lot from this project and from your suggestions. @NMV01 and I, we will be happy to share ideas and thoughts in future. Regards, |
As we discussed before, this PR comes from splitting PR #357 into two pull requests (the other PR is #364).
The main goal is to add SSRF and DoS vulnerabilities based on the user cases defined before.
Modifications like making some methods private or adding the contributor's name, have been made. We didn't change our implementation.