-
-
Notifications
You must be signed in to change notification settings - Fork 262
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
CPU usage #26
Comments
You say old script before this update what update version are you comparing to. bf420a3 It will help me identify what is causing CPU hogging. |
Sorry for misunderstanding, old script - another code, not related to your repository. |
@rx-209 I know your issue was unrelated to my code as you said but I went ahead and Optimized / squeezed my code for all the performance it can give you with Nginx and LuaJIT including having it cache regex. If you want something i use in production on my own Nginx servers turn this feature on http://nginx.org/en/docs/ngx_core_module.html#pcre_jit |
Great, cpu load decreased by 1,5-2 times, thx. |
@rx-209 No problem glad you see a huge performance impact i will tweak it further to get the last couple of things upto standard but you most likely won't see a performance gain because it will be such a small one it won't be noticeable. |
Now using script for real ddos mitigation. When i instaled your script insted of my old code - cpu usage highly increased.
The text was updated successfully, but these errors were encountered: