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

Allow selection of an input ID for the simulation message #78

Merged
merged 2 commits into from Aug 10, 2016

Conversation

Projects
None yet
2 participants
@bernd
Member

bernd commented Aug 10, 2016

  1. Invert equals() call to avoid possible null pointer exception.
    Fixes Graylog2/graylog2-server#2610
  2. Pipeline rules may use the from_input function in a condition so this
    is needed to make the simulation work.
    Refs Graylog2/graylog2-server#2610

bernd added some commits Aug 10, 2016

Allow selection of an input ID for the simulation message
Pipeline rules may use the `from_input` function in a condition so this
is needed to make the simulation work.

Refs Graylog2/graylog2-server#2610

@kroepke kroepke self-assigned this Aug 10, 2016

@@ -72,7 +72,7 @@ public Boolean evaluate(FunctionArgs args, EvaluationContext context) {

}
return input != null
&& context.currentMessage().getSourceInputId().equals(input.getId());
&& input.getId().equals(context.currentMessage().getSourceInputId());

This comment has been minimized.

@kroepke
@kroepke

This comment has been minimized.

Member

kroepke commented Aug 10, 2016

lgtm

@kroepke kroepke merged commit 567256a into master Aug 10, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@kroepke kroepke deleted the server-issue-2610 branch Aug 10, 2016

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