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

Added vhost options SecRuleRemoveByTag and SecRuleRemoveByMsg #1407

Merged
merged 2 commits into from Apr 14, 2016
Merged

Added vhost options SecRuleRemoveByTag and SecRuleRemoveByMsg #1407

merged 2 commits into from Apr 14, 2016

Conversation

FlatKey
Copy link
Contributor

@FlatKey FlatKey commented Mar 26, 2016

  • documentation included
  • tests included
  • all tests successfully passed (rake spec, rspec spec/acceptance)

@FlatKey FlatKey mentioned this pull request Mar 26, 2016
Closed
@jonnytdevops
Copy link
Contributor

@FlatKey it would be appreciated if you could rebase this against current master as changes have gone in that should fix the failing unit tests (assuming nothing in this PR breaks them).

Thanks

@FlatKey FlatKey force-pushed the ModSec-SecRuleRemoveBy-Tag-and-Msg branch from ff870ee to 800c7ce Compare March 30, 2016 19:22
@FlatKey
Copy link
Contributor Author

FlatKey commented Mar 30, 2016

@jonnytpuppet rebased.

@bmjen bmjen merged commit 1a2e229 into puppetlabs:master Apr 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@FlatKey @jonnytdevops @bmjen @GSPatton