Skip to content
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

Modify rule IDs for whitelisting rules #30

Closed
studersi opened this issue Mar 30, 2018 · 2 comments
Closed

Modify rule IDs for whitelisting rules #30

studersi opened this issue Mar 30, 2018 · 2 comments
Assignees

Comments

@studersi
Copy link
Contributor

Unclear Step 8: Writing simple whitelist rules

Not clear that the rules introduced in this step replace the rule added in
step 7. The only indication is that the rule ID 10000 appears twice.

162 of /opt/apache-2.4.29/conf/httpd.conf: ModSecurity: Found another rule
with the same id ```

Yes, I'm aware of that. I have a note here telling me to move the whitelisting
rules +1000 to make sure there are no longer any collisions.

@dune73
Copy link
Owner

dune73 commented Apr 3, 2018

Change performed in commit

e13a443

Could you please check this works as intended?

@studersi
Copy link
Contributor Author

studersi commented May 8, 2018

Your commit creates a new collision later in the tutorial. It is addressed in PR #83.

@studersi studersi closed this as completed May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants