You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Claus Ibsen (JIRA)" <ji...@apache.org> on 2015/05/27 14:48:21 UTC

[jira] [Commented] (CAMEL-8799) Make it possible for JsonPath to suppress PathNotFoundException

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

Claus Ibsen commented on CAMEL-8799:
------------------------------------

Ideally we should do #2. But your PR for #1 can likely be reused when doing #2 such as unit tests etc. You are welcome to do the PR.

The DSL needs to be updated in camel-core which is just exposed in
https://github.com/apache/camel/blob/master/camel-core/src/main/java/org/apache/camel/model/language/JsonPathExpression.java

And then likely a change needed to make fluent builders for Java DSL as well. 

> 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)