Merge pull request #91 from Spikhalskiy/jaxrs1-compatibility

Implemented dynamic selection of NoContentException provider to back support of JaxRS 1.x

base/src/main/java/com/fasterxml/jackson/jaxrs/base/NoContentExceptionSupplier.java

@@ -0,0 +1,13 @@
+package com.fasterxml.jackson.jaxrs.base;
+
+import java.io.IOException;
+
+/**
+ * Implementors of this class contains strategies for NoContentException creation
+ */
+public interface NoContentExceptionSupplier
+{
+    String NO_CONTENT_MESSAGE = "No content (empty input stream)";
+
+    IOException createNoContentException();
+}

base/src/main/java/com/fasterxml/jackson/jaxrs/base/ProviderBase.java

@@ -2,6 +2,7 @@
 
 import java.io.*;
 import java.lang.annotation.Annotation;
+import java.lang.reflect.Constructor;
 import java.lang.reflect.Type;
 import java.util.*;
 import java.util.concurrent.atomic.AtomicReference;
@@ -13,6 +14,8 @@
 import com.fasterxml.jackson.core.*;
 import com.fasterxml.jackson.databind.*;
 import com.fasterxml.jackson.databind.type.TypeFactory;
+import com.fasterxml.jackson.jaxrs.base.nocontent.JaxRS1NoContentExceptionSupplier;
+import com.fasterxml.jackson.jaxrs.base.nocontent.JaxRS2NoContentExceptionSupplier;
 import com.fasterxml.jackson.jaxrs.cfg.*;
 import com.fasterxml.jackson.jaxrs.util.ClassKey;
 import com.fasterxml.jackson.jaxrs.util.LRUMap;
@@ -33,6 +36,10 @@ public abstract class ProviderBase<
      */
     public final static String HEADER_CONTENT_TYPE_OPTIONS = "X-Content-Type-Options";
 
+    protected final static String CLASS_NAME_NO_CONTENT_EXCEPTION = "javax.ws.rs.core.NoContentException";
+
+    protected final NoContentExceptionSupplier noContentExceptionSupplier = _createNoContentExceptionSupplier();
+
     /**
      * Looks like we need to worry about accidental
      *   data binding for types we shouldn't be handling. This is
@@ -967,9 +974,7 @@ protected boolean _isIgnorableForWriting(ClassKey typeKey)
      */
     protected IOException _createNoContentException()
     {
-        // 29-Jun-2016, tatu: With Jackson 2.8 we require JAX-RS 2.0 so this
-        //    is fine; earlier had complicated Reflection-based access
-        return new NoContentException("No content (empty input stream)");
+        return noContentExceptionSupplier.createNoContentException();
     }
 
     /*
@@ -1046,4 +1051,24 @@ protected static void _addSuperTypes(Class<?> cls, Class<?> endBefore, Collectio
     private final THIS _this() {
         return (THIS) this;
     }
+
+    /**
+     * Since class <code>javax.ws.rs.core.NoContentException</code> only exists in
+     * JAX-RS 2.0, but we want to have 1.x compatibility, need to dynamically select exception supplier
+     */
+    private static NoContentExceptionSupplier _createNoContentExceptionSupplier() {
+        try {
+            Class cls = Class.forName(CLASS_NAME_NO_CONTENT_EXCEPTION);
+            Constructor<?> ctor = cls.getDeclaredConstructor(String.class);
+            if (ctor != null) {
+                return new JaxRS2NoContentExceptionSupplier();
+            } else {
+                return new JaxRS1NoContentExceptionSupplier();
+            }
+        } catch (ClassNotFoundException ex) {
+            return new JaxRS1NoContentExceptionSupplier();
+        } catch (NoSuchMethodException e) {
+            return new JaxRS1NoContentExceptionSupplier();
+        }
+    }
 }

base/src/main/java/com/fasterxml/jackson/jaxrs/base/nocontent/JaxRS1NoContentExceptionSupplier.java

@@ -0,0 +1,18 @@
+package com.fasterxml.jackson.jaxrs.base.nocontent;
+
+import com.fasterxml.jackson.jaxrs.base.NoContentExceptionSupplier;
+
+import java.io.IOException;
+
+/**
+ * Create plain IOException for JaxRS 1.x because {@link javax.ws.rs.core.NoContentException}
+ * has been introduced in JaxRS 2.x
+ */
+public class JaxRS1NoContentExceptionSupplier implements NoContentExceptionSupplier
+{
+    @Override
+    public IOException createNoContentException()
+    {
+        return new IOException(NO_CONTENT_MESSAGE);
+    }
+}

base/src/main/java/com/fasterxml/jackson/jaxrs/base/nocontent/JaxRS2NoContentExceptionSupplier.java

@@ -0,0 +1,16 @@
+package com.fasterxml.jackson.jaxrs.base.nocontent;
+
+import com.fasterxml.jackson.jaxrs.base.NoContentExceptionSupplier;
+
+import javax.ws.rs.core.NoContentException;
+import java.io.IOException;
+
+/**
+ * This supplier creates fair NoContentException from JaxRS 2.x
+ */
+public class JaxRS2NoContentExceptionSupplier implements NoContentExceptionSupplier {
+    @Override
+    public IOException createNoContentException() {
+        return new NoContentException(NoContentExceptionSupplier.NO_CONTENT_MESSAGE);
+    }
+}