camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CAMEL-8799) Make it possible for JsonPath to suppress PathNotFoundException
Date Wed, 27 May 2015 14:35:23 GMT

    [ https://issues.apache.org/jira/browse/CAMEL-8799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14561079#comment-14561079
] 

ASF GitHub Bot commented on CAMEL-8799:
---------------------------------------

GitHub user pax95 opened a pull request:

    https://github.com/apache/camel/pull/528

    CAMEL-8799: Make json-path suppress PathNotFoundExceptions by default

    This PR is a solution for option 1 described in the issue, and the tests can be reused
in the implementation of option 2.

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

    $ git pull https://github.com/pax95/camel master

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

    https://github.com/apache/camel/pull/528.patch

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

    This closes #528
    
----
commit e09bd00d9199c88787f0acb0d2ea8c86e6a1bba9
Author: Preben Asmussen <preben.asmussen@gmail.com>
Date:   2015-05-26T15:37:28Z

    Make json-path suppress PathNotFoundExceptions by default

----


> Make it possible for JsonPath to suppress PathNotFoundException
> ---------------------------------------------------------------
>
>                 Key: CAMEL-8799
>                 URL: https://issues.apache.org/jira/browse/CAMEL-8799
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-jsonpath
>    Affects Versions: 2.15.2
>            Reporter: Preben Asmussen
>             Fix For: 2.16.0
>
>
> It should be possible to suppress exceptions when evaluating JsonPath's.
> See http://camel.465427.n5.nabble.com/How-to-suppress-exceptions-from-Jsonpath-expressions-td5767384.html
> There are 2 options
> 1.  Make it the default behaviour, so a null value will be returned instead of throwing
an PathNotFoundException when evaluating json path expressions and the path is not found.
I have a working patch for that here https://github.com/pax95/camel/commit/e09bd00d9199c88787f0acb0d2ea8c86e6a1bba9
that I can do a PR on.
> 2. Make it configurable on the JsonPath component so you can enable/disable it on demand.
This would also make it possible to expose other options from com.jayway.jsonpath.Option if
they make sense. More work is require to make it available as both xml and java dsl.
> Currently I'm at bit overloaded to do a PR on 2 if that is the preferred option, so if
any want to take over It will be fine.
> If option 1 is ok I'll create a PR.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message