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.

Sign up for GitHub

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

Jump to bottom

fix: Read response body more efficiently #4

Closed

erikgeiser wants to merge 2 commits into RedTeamPentesting:master from erikgeiser:fix/fix-bottleneck

Contributor

erikgeiser commented Sep 14, 2020

This PR removes the (for most request unnecessary) allocation of 5MB for the response body per request.

I tested this with a 45k-lines wordlist against a local HTTPS test server in Go that returns a random status code and small response bodies:

Branch	Requests/s	Time
Current Master	1811	19s
This PR	11947	3s

erikgeiser added 2 commits

September 14, 2020 21:16


          fix: Read response body more efficiently

c30698e


          Rename body buffer size

c9f3045

Member

rtpt-alexanderneumann commented Sep 16, 2020

Thanks! Merged as e051791

rtpt-alexanderneumann closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment