[ 
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 <[email protected]>
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)

Reply via email to