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

Add contains string tester #3186

Merged
merged 1 commit into from Dec 13, 2016

Conversation

Projects
None yet
2 participants
@edmundoa
Member

edmundoa commented Dec 12, 2016

Introducing a new contains string tester that let us check if an extractor would run against a message when it contains a string. So far we were using the regex tester, but that requires to escape some characters.

Fixes #3134

Add contains string tester
We should not use the regex tester to verify that a string contains
another string, as it requires to escape some characters.

Fixes #3134

@edmundoa edmundoa added this to the 2.2.0 milestone Dec 12, 2016

@joschi joschi self-requested a review Dec 12, 2016

@joschi joschi self-assigned this Dec 12, 2016

@joschi

joschi approved these changes Dec 13, 2016

@joschi joschi merged commit 0956961 into master Dec 13, 2016

4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1128 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
licence/cla Contributor License Agreement is signed.
Details

@joschi joschi deleted the issue-3134 branch Dec 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment