brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aledsage <>
Subject [GitHub] brooklyn-server pull request #793: HttpRequestSensor: handling non-json
Date Fri, 04 Aug 2017 18:24:38 GMT
GitHub user aledsage opened a pull request:

    HttpRequestSensor: handling non-json

    Previously the `HttpRequestSensor` failed if no `jsonPath: ` is suppiled in the yaml.
I've added a failing test, but not worked specifically on that. It doesn't pick up the default
value of `$`.
    Added support for setting `jsonPath` to blank, so that it just takes the pure-string value
(i.e. no json parsing).
    Also added a few more (unrelated!) tests.

You can merge this pull request into a Git repository by running:

    $ git pull httpRequest-handling-nonJson

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #793
commit b571592d86dcf8fbf35db0aa6b473ed5cb0e990c
Author: Aled Sage <>
Date:   2017-08-04T18:14:57Z

    HttpRequestSensor: handling non-json

commit 5d18f57e816d225fecc4c7b902e4c7c63d0f938a
Author: Aled Sage <>
Date:   2017-08-04T18:18:47Z

    Add testListExcludesBlank for aggregrator

commit 196c40d1fc597fdfe5e0423515cce2bc2396fbe9
Author: Aled Sage <>
Date:   2017-08-04T18:19:01Z

    Adds PropagatorYamlTest


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message