Skip to content
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

[FEATURE REQUEST] Detect excessive 403s #200

Closed
N0ur5 opened this issue Jan 25, 2021 · 2 comments · Fixed by #221
Closed

[FEATURE REQUEST] Detect excessive 403s #200

N0ur5 opened this issue Jan 25, 2021 · 2 comments · Fixed by #221
Labels
enhancement New feature or request has-PR pinned
Milestone

Comments

@N0ur5
Copy link

N0ur5 commented Jan 25, 2021

Hello Friend,

I have been very happy with the 403 directory recursion feature you added after our discussion about it. Thank you again for that. Today for the first time however it backfired on me a little (not a huge deal but worth mentioning).

The IP I was scanning a web app from got blacklisted while I had ferox running in the background. As a result, every single request generated a 403 response. This quickly turned every word in my wordlist into a 403'd dir which was added to the array of dirs to be recursive with. My terminals were locked up for a good 15 minutes until my crtl+c caught up with things.

So my thought is, perhaps if some number of consecutive 403's are hit, the scan gets auto-paused and the user is prompted something to the effect of "Excessive 403 detected, you may be blacklisted, continue anyways? (Y/n)".

In this situation I can ask the devs to whitelist me for further scanning so all is well, but I might not be the last person to have this happen if I don't bring it up!

Thanks!

@epi052 epi052 added enhancement New feature or request pinned labels Jan 26, 2021
@epi052 epi052 changed the title Rethinking the 403 recursion [FEATURE REQUEST] Detect excessive 403s Jan 26, 2021
@epi052
Copy link
Owner

epi052 commented Jan 26, 2021

Hey there!

Thanks for the report. This is incredibly similar to #123. I've already got all the mechanics of tracking 403s (among other things) and plan to monitor for excessive errors (403s included) and either reduce request rate or bail completely.

I'm actively rewriting large chunks of the program right now for a 2.0.0 release. Some of the changes in 2.0.0 will help move this and #123 closer to completion. Pinning for now, but not likely to work on this and #123 until after 2.0.0 is released.

@epi052
Copy link
Owner

epi052 commented Feb 4, 2021

progress on this issue can be seen at #212

@epi052 epi052 added the has-PR label Feb 4, 2021
@epi052 epi052 mentioned this issue Feb 6, 2021
13 tasks
@epi052 epi052 added this to the 2.1.0 milestone Feb 14, 2021
@epi052 epi052 mentioned this issue Feb 17, 2021
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request has-PR pinned
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants