-
Notifications
You must be signed in to change notification settings - Fork 1.7k
DOCS: Still talking about ModSecurityPass #1075
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
Comments
So is the suggestion just to change the order? |
Well, ideally you would remove the old-version content. ;-) But if you don't want to do that, then surely you should be promoting the new version and putting old-version config stuff in small print ? |
I'll probably add a note - do you remember what version we removed ModSecurityPass in? |
2.7.2 I believe. |
it appears you are correct. |
Alright @udf2457 I have updated the wiki tell me if you think this reads better. Thank you for your feedback we do appreciate fixes like this :) |
@csanders-git beautiful, thank you ! |
Hi,
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#Installation_for_NGINX (configuration steps).
Still banging on about ModSecurityPass despite "remove Nginx ModSecurityPass command" in #27
(oh, I see you mention the new way below the old way... perhaps you should consider inverting it ;-)
The text was updated successfully, but these errors were encountered: