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

Ensures watch definitions are valid json #30692

Closed
wants to merge 5 commits into from
Closed

Ensures watch definitions are valid json #30692

wants to merge 5 commits into from

Commits on May 17, 2018

  1. Ensures watch definitions are valid json 

    Currently, watches may be submitted and accepted by the cluster
that do not consist of valid JSON.  This is because in certain
cases, WatchParser.java will not consume enough tokens from
XContentParser to be able to encounter an underlying JSON parse
error.  This patch fixes this behavior and adds a test.

Closes elastic#29746
    Thomas Callahan authored and Thomas Callahan committed May 17, 2018
    Configuration menu
    Copy the full SHA
    36d0c12 View commit details
    Browse the repository at this point in the history

  2. address reviewer feedback

    Thomas Callahan authored and Thomas Callahan committed May 17, 2018
    Configuration menu
    Copy the full SHA
    6da9e8b View commit details
    Browse the repository at this point in the history

  3. address reviewer feedback

    Thomas Callahan authored and Thomas Callahan committed May 17, 2018
    Configuration menu
    Copy the full SHA
    7252b00 View commit details
    Browse the repository at this point in the history

Commits on May 23, 2018

  1. address reviewer feedback

    @tomcallahan
    tomcallahan committed May 23, 2018
    Configuration menu
    Copy the full SHA
    9a4db9f View commit details
    Browse the repository at this point in the history

Commits on May 24, 2018

  1. Merge remote-tracking branch 'origin/master' into ensure-watch-valid-… 

    …json
    @tomcallahan
    tomcallahan committed May 24, 2018
    Configuration menu
    Copy the full SHA
    dcd9b86 View commit details
    Browse the repository at this point in the history