-
Notifications
You must be signed in to change notification settings - Fork 250
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
fix: improvements in memory consumption #100
Conversation
I think it should help to close #80
- preallocate config slices - not used interfaces in consumer - do not allocate new variables in loops
Hi! The module name has been changed, it doesn't matter, and if there are any problems, let me know, I'll return it back. The socks5 library used has been changed to a more up-to-date one. We also made a small optimization on the use of variables in loops. As a result, much less memory is consumed and requests are processed faster. |
Thanks for this huge PR. Really appreciate it. I'd review it when I have more bandwidth. Is this PR ready or are there more things to fix? |
I just corrected one of the errors that caused multiple http connections to be handled incorrectly. Now I think that everything is correct and can be checked. |
I think it should help to close #80