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.

Sign up for GitHub

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

Jump to bottom

00-load-modsec.t test case is disregarding others that are not Apache. #643

Closed
zimmerle opened this issue Jan 9, 2014 · 1 comment
Closed

00-load-modsec.t test case is disregarding others that are not Apache. #643

zimmerle opened this issue Jan 9, 2014 · 1 comment
Labels
Platform - Nginx
Milestone
v2.8.0-RC1

Comments

@zimmerle
Copy link
Contributor

zimmerle commented Jan 9, 2014

It is trying to match "ModSecurity for Apache.* configured". It should try to match "ModSecurity for nginx.* configured", in case the nginx version is being used/tested.
@zimmerle zimmerle mentioned this issue Jan 9, 2014
Closed
@zimmerle
Copy link
Contributor Author

zimmerle commented Jan 9, 2014

By 7478faa we have the ability to match different content depending on the verison/port. Using this functionality to correct handle that simple use case, committed at: 3cf1701

@zimmerle zimmerle added this to the testing milestones milestone Mar 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Platform - Nginx
Projects
None yet
Milestone
v2.8.0-RC1
Development

No branches or pull requests
1 participant
@zimmerle