This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/master by this push: new b59a96d CAMEL-16389 - Remove json-smart b59a96d is described below commit b59a96ddf1ba647b979f937f5bd956bba32173ee Author: Colm O hEigeartaigh <cohei...@apache.org> AuthorDate: Tue Mar 23 12:49:44 2021 +0000 CAMEL-16389 - Remove json-smart --- camel-dependencies/pom.xml | 1 - components/camel-chunk/pom.xml | 5 ----- components/camel-jsonpath/pom.xml | 9 +++++++-- .../src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java | 4 ++++ .../java/org/apache/camel/jsonpath/JsonPathMapTransformTest.java | 7 ++++++- parent/pom.xml | 1 - 6 files changed, 17 insertions(+), 10 deletions(-) diff --git a/camel-dependencies/pom.xml b/camel-dependencies/pom.xml index a485e98..f8cc99a 100644 --- a/camel-dependencies/pom.xml +++ b/camel-dependencies/pom.xml @@ -361,7 +361,6 @@ <json-path-version>2.4.0</json-path-version> <json-schema-validator-version>2.2.14</json-schema-validator-version> <json-simple-version>1.1.1</json-simple-version> - <json-smart-version>2.3</json-smart-version> <jsonassert-version>1.5.0</jsonassert-version> <jsonata4java-version>1.5.0</jsonata4java-version> <jsoup-version>1.13.1</jsoup-version> diff --git a/components/camel-chunk/pom.xml b/components/camel-chunk/pom.xml index 41f63c1..71bc279 100644 --- a/components/camel-chunk/pom.xml +++ b/components/camel-chunk/pom.xml @@ -46,11 +46,6 @@ <version>${chunk-templates-version}</version> </dependency> <dependency> - <groupId>net.minidev</groupId> - <artifactId>json-smart</artifactId> - <version>${json-smart-version}</version> - </dependency> - <dependency> <groupId>commons-io</groupId> <artifactId>commons-io</artifactId> <version>${commons-io-version}</version> diff --git a/components/camel-jsonpath/pom.xml b/components/camel-jsonpath/pom.xml index 60f2aa8..7007ab7 100644 --- a/components/camel-jsonpath/pom.xml +++ b/components/camel-jsonpath/pom.xml @@ -34,7 +34,7 @@ <properties> <camel.osgi.import> - com.fasterxml.jackson.databind;resolution:=optional, + com.fasterxml.jackson.databind, com.fasterxml.jackson.module.jaxb;resolution:=optional, * </camel.osgi.import> @@ -55,11 +55,16 @@ <groupId>com.jayway.jsonpath</groupId> <artifactId>json-path</artifactId> <version>${json-path-version}</version> + <exclusions> + <exclusion> + <groupId>net.minidev</groupId> + <artifactId>json-smart</artifactId> + </exclusion> + </exclusions> </dependency> <dependency> <groupId>com.fasterxml.jackson.core</groupId> <artifactId>jackson-databind</artifactId> - <optional>true</optional> </dependency> <dependency> <groupId>com.fasterxml.jackson.module</groupId> diff --git a/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java b/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java index 40be969..d64eca0f 100644 --- a/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java +++ b/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java @@ -30,6 +30,8 @@ import java.util.regex.Pattern; import com.jayway.jsonpath.Configuration; import com.jayway.jsonpath.JsonPath; import com.jayway.jsonpath.Option; +import com.jayway.jsonpath.spi.json.JacksonJsonProvider; +import com.jayway.jsonpath.spi.mapper.JacksonMappingProvider; import org.apache.camel.CamelExchangeException; import org.apache.camel.Exchange; import org.apache.camel.Expression; @@ -72,6 +74,8 @@ public class JsonPathEngine { if (options != null) { builder.options(options); } + builder.jsonProvider(new JacksonJsonProvider()); + builder.mappingProvider(new JacksonMappingProvider()); if (suppressExceptions) { builder.options(SUPPRESS_EXCEPTIONS); } diff --git a/components/camel-jsonpath/src/test/java/org/apache/camel/jsonpath/JsonPathMapTransformTest.java b/components/camel-jsonpath/src/test/java/org/apache/camel/jsonpath/JsonPathMapTransformTest.java index 7c2230f..61af6ea 100644 --- a/components/camel-jsonpath/src/test/java/org/apache/camel/jsonpath/JsonPathMapTransformTest.java +++ b/components/camel-jsonpath/src/test/java/org/apache/camel/jsonpath/JsonPathMapTransformTest.java @@ -21,6 +21,8 @@ import java.util.List; import java.util.Map; import com.jayway.jsonpath.Configuration; +import com.jayway.jsonpath.spi.json.JacksonJsonProvider; +import com.jayway.jsonpath.spi.mapper.JacksonMappingProvider; import org.apache.camel.builder.RouteBuilder; import org.apache.camel.test.junit5.CamelTestSupport; import org.junit.jupiter.api.Test; @@ -47,7 +49,10 @@ public class JsonPathMapTransformTest extends CamelTestSupport { getMockEndpoint("mock:authors").expectedMessageCount(1); // should be a map - Object document = Configuration.defaultConfiguration().jsonProvider() + Configuration.ConfigurationBuilder builder = Configuration.builder(); + builder.jsonProvider(new JacksonJsonProvider()); + builder.mappingProvider(new JacksonMappingProvider()); + Object document = builder.build().jsonProvider() .parse(new FileInputStream("src/test/resources/books.json"), "utf-8"); assertIsInstanceOf(Map.class, document); diff --git a/parent/pom.xml b/parent/pom.xml index 168d6b12..0603fe8 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -345,7 +345,6 @@ <jsonassert-version>1.5.0</jsonassert-version> <json-path-version>2.4.0</json-path-version> <json-simple-version>1.1.1</json-simple-version> - <json-smart-version>2.3</json-smart-version> <jsonata4java-version>1.5.0</jsonata4java-version> <json-schema-validator-version>2.2.14</json-schema-validator-version> <jsoup-version>1.13.1</jsoup-version>