- Made JacksonJSonParser extend ObjectMapper instead of using it.

- Made response from server on errors be displayed under debug log instead of info.

Author: gastonortiz

README.md

@@ -1,6 +1,7 @@
 # Java Client Library (beta)
 
-### Latest released version v0.7.0
+### Latest released version v0.7.1
+### Latest snapshot version v0.7.2-SNAPSHOT
 
 ## Introduction
 Welcome my friends! This is the Poly API Java client GitHub page. If you are here, then it means you're familiar with what we do at Poly. If you aren't, you can always check [here](https://github.com/polyapi/poly-alpha).
@@ -54,7 +55,7 @@ Nice to have some customers looking around here! So, you'll need to run the foll
 2. **Update the project.** Add the following to your project's `pom.xml`:
 ```xml
 <properties>
-  <poly.version>0.7.0</poly.version>
+  <poly.version>0.7.1</poly.version>
 </properties>
 <dependencies>
   <dependency>
@@ -314,6 +315,9 @@ Comparing to its Typescript counterpart, the Java library is still missing the f
 These features will be added in the future releases.
 
 ## Changelog
+### v0.7.2
+- Made JacksonJSonParser extend ObjectMapper instead of using it.
+- Made response from server on errors be displayed under debug log instead of info.
 ### v0.7.1
 - Fixed bug about having duplicate schema types with the same name.
 - Improved minor code issues.

commons/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>io.polyapi</groupId>
         <artifactId>parent-pom</artifactId>
-        <version>0.8.0-SNAPSHOT</version>
+        <version>0.7.2-SNAPSHOT</version>
         <relativePath>../parent-pom</relativePath>
     </parent>
 

commons/src/main/java/io/polyapi/commons/internal/json/JacksonJsonParser.java

@@ -25,15 +25,15 @@
  * Wrapper class around the Jackson mapping library to handle all the errors it can throw and unify all the configuration for it.
  */
 @Slf4j
-public class JacksonJsonParser implements JsonParser {
-    private final ObjectMapper objectMapper;
+public class JacksonJsonParser extends ObjectMapper implements JsonParser {
     private final JsonSchemaGenerator jsonSchemaGenerator;
 
     /**
      * Utility constructor that uses a standard {@link ObjectMapper} instance.
      */
     public JacksonJsonParser() {
-        this(new ObjectMapper().configure(FAIL_ON_UNKNOWN_PROPERTIES, false));
+        configure(FAIL_ON_UNKNOWN_PROPERTIES, false);
+        this.jsonSchemaGenerator = new JsonSchemaGenerator(this);
     }
 
     /**
@@ -42,7 +42,7 @@ public JacksonJsonParser() {
      * @param objectMapper The object mapper.
      */
     public JacksonJsonParser(ObjectMapper objectMapper) {
-        this.objectMapper = objectMapper;
+        super(objectMapper);
         this.jsonSchemaGenerator = new JsonSchemaGenerator(objectMapper);
     }
 
@@ -53,7 +53,7 @@ public JacksonJsonParser(ObjectMapper objectMapper) {
     public String toJsonString(Object object) {
         try {
             log.debug("Parsing object of type {} to String.", object.getClass().getSimpleName());
-            String result = objectMapper.writeValueAsString(object);
+            String result = writeValueAsString(object);
             log.debug("Object to String parsing successful.");
             if (log.isTraceEnabled()) {
                 log.trace("Parsed result is:\n{}", result);
@@ -71,7 +71,7 @@ public String toJsonString(Object object) {
     public InputStream toJsonInputStream(Object object) {
         try {
             log.debug("Parsing object of type {} to InputStream.", Optional.ofNullable(object).map(Object::getClass).map(Class::getName).orElse("null"));
-            InputStream result = new ByteArrayInputStream(object == null ? new byte[]{} : objectMapper.writeValueAsBytes(object));
+            InputStream result = new ByteArrayInputStream(object == null ? new byte[]{} : writeValueAsBytes(object));
             log.debug("String to object parsing successful.");
             if (log.isTraceEnabled()) {
                 log.trace("Parsed result is:\n{}", IOUtils.toString(result, defaultCharset()));
@@ -95,7 +95,7 @@ public <O> O parseString(String json, Type expectedResponseType) {
                 log.trace("Input to parse is:\n{}", json);
             }
             log.debug("Parsing JSon String to object of type {}.", expectedResponseType.getTypeName());
-            O result = objectMapper.readValue(json, defaultInstance().constructType(expectedResponseType));
+            O result = readValue(json, defaultInstance().constructType(expectedResponseType));
             log.debug("Parsing successful.");
             return result;
         } catch (IOException e) {
@@ -124,7 +124,7 @@ public <O> O parseInputStream(InputStream json, Type expectedResponseType) {
             if (expectedResponseType == String.class) {
                 result = (O) IOUtils.toString(json, Charset.defaultCharset());
             } else {
-                result = objectMapper.readValue(json, defaultInstance().constructType(expectedResponseType));
+                result = readValue(json, defaultInstance().constructType(expectedResponseType));
             }
 
             log.debug("Parsing successful.");

library/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>io.polyapi</groupId>
         <artifactId>parent-pom</artifactId>
-        <version>0.8.0-SNAPSHOT</version>
+        <version>0.7.2-SNAPSHOT</version>
         <relativePath>../parent-pom</relativePath>
     </parent>
     <artifactId>library</artifactId>

parent-pom/pom.xml

@@ -3,7 +3,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>io.polyapi</groupId>
     <artifactId>parent-pom</artifactId>
-    <version>0.8.0-SNAPSHOT</version>
+    <version>0.7.2-SNAPSHOT</version>
     <packaging>pom</packaging>
     <name>Poly API Java parent POM</name>
     <url>https://polyapi.io</url>

polyapi-maven-plugin/pom.xml

@@ -4,7 +4,7 @@
     <parent>
         <groupId>io.polyapi</groupId>
         <artifactId>parent-pom</artifactId>
-        <version>0.8.0-SNAPSHOT</version>
+        <version>0.7.2-SNAPSHOT</version>
         <relativePath>../parent-pom</relativePath>
     </parent>
     <artifactId>polyapi-maven-plugin</artifactId>

polyapi-maven-plugin/src/main/java/io/polyapi/plugin/mojo/PolyApiMojo.java

@@ -70,13 +70,13 @@ public void execute() throws MojoExecutionException, MojoFailureException {
             throw new MojoFailureException(e);
         } catch (UnexpectedHttpResponseException e) {
             log.error("An unexpected HTTP response code {} was returned from the server.", e.getResponse().statusCode());
-            //if (log.isTraceEnabled()) {
+            if (log.isDebugEnabled()) {
                 try {
-                    log.info("Response from server is: {}", IOUtils.toString(e.getResponse().body(), defaultCharset()));
+                    log.debug("Response from server is: {}", IOUtils.toString(e.getResponse().body(), defaultCharset()));
                 } catch (IOException ex) {
                     throw new MojoExecutionException(ex);
                 }
-            //}
+            }
             throw new MojoFailureException(e);
         } catch (RuntimeException e) {
             log.error("An unexpected exception occurred during the plugin execution.", e);

pom.xml

@@ -3,7 +3,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>io.polyapi</groupId>
     <artifactId>polyapi-java</artifactId>
-    <version>0.8.0-SNAPSHOT</version>
+    <version>0.7.2-SNAPSHOT</version>
     <packaging>pom</packaging>
     <modules>
         <module>parent-pom</module>