-
Notifications
You must be signed in to change notification settings - Fork 52
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
Adding Modsecurity #13
Conversation
Thank you for the PR. A few things to correct:
|
ModSecurity checked out from a stable tag. Added NGINX_DISABLE_MODSECURITY var to disable loading of module Removed and dynamically added the file /etc/nginx/modsec/main.conf Updated Read me to mention the module and OWASP
I've just noticed you also install packages that are not deleted:
are those required runtime packages? do we really need all of them? |
@csandanov Hi , Sorry that should have been moved to the other modsecurity build deps, which will get removed. I tested it and also added some more configuration options. |
Are you sure none of the packages needed in runtime? Could you please create a new PR and describe what are the cases when it breaks something and why, I know very little about mod_security configuration. Thank you |
Adding mod security module to nginx with default owas common rule set from https://github.com/SpiderLabs/owasp-modsecurity-crs/