Fix #1368

Author: cowtowncoder

release-notes/CREDITS

@@ -471,3 +471,8 @@ Brian Pontarelli (voidmain@github)
 Max Drobotov (fizmax@github)
   * Reported, contributed fix for #1332: `ArrayIndexOutOfBoundException` for enum by index deser
    (2.7.7)
+
+Josh Caplan (jecaplan@github)
+  * Reported, suggested fix for #1368: Problem serializing `JsonMappingException` due to addition
+   of non-ignored `processor` property (added in 2.7)
+  (2.7.8)

release-notes/VERSION

@@ -10,6 +10,9 @@ Project: jackson-databind
 #1359: Improve `JsonNode` deserializer to create `FloatNode` if parser supports
 #1362: ObjectReader.readValues()` ignores offset and length when reading an array
  (reported by wastevenson@github)
+#1368: Problem serializing `JsonMappingException` due to addition of non-ignored
+  `processor` property (added in 2.7)
+ (reported, suggesed fix by Josh C)
 
 2.7.7 (27-Aug-2016)
 

src/main/java/com/fasterxml/jackson/databind/JsonMappingException.java

@@ -5,6 +5,7 @@
 import java.io.Serializable;
 import java.util.*;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.core.*;
 import com.fasterxml.jackson.databind.util.ClassUtil;
 
@@ -400,7 +401,7 @@ public List<Reference> getPath()
     }
 
     /**
-     * Method for accesing description of path that lead to the
+     * Method for accessing description of path that lead to the
      * problem that triggered this exception
      */
     public String getPathReference()
@@ -454,6 +455,7 @@ public void prependPath(Reference r)
      */
 
     @Override // since 2.7.5
+    @JsonIgnore // as per [databind#1368]
     public Object getProcessor() { return _processor; }
 
     @Override

src/main/java/com/fasterxml/jackson/databind/ser/BeanSerializerFactory.java

@@ -587,6 +587,7 @@ protected List<BeanPropertyWriter> findBeanProperties(SerializerProvider prov,
         ArrayList<BeanPropertyWriter> result = new ArrayList<BeanPropertyWriter>(properties.size());
         final boolean fixAccess = config.canOverrideAccessModifiers();
         final boolean forceAccess = fixAccess && config.isEnabled(MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS);
+
         for (BeanPropertyDefinition property : properties) {
             final AnnotatedMember accessor = property.getAccessor();
             // Type id? Requires special handling:

src/test/java/com/fasterxml/jackson/databind/deser/TestExceptionDeserialization.java

@@ -136,4 +136,33 @@ public void testSingleValueArrayDeserializationException() throws Exception {
             //Exception thrown correctly
         }
     }
+
+    static class NoSerdeConstructor {
+        private String strVal;
+        public String getVal() { return strVal; }
+        public NoSerdeConstructor( String strVal ) {
+            this.strVal = strVal;
+        }
+    }
+
+    // [databind#1368]
+    public void testFailingNoSerdeConstructor() throws IOException {
+        Exception e = null;
+        // cant deserialize due to unexpected constructor
+        try {
+            MAPPER.readValue( "{ \"val\": \"foo\" }", NoSerdeConstructor.class );
+            fail("Should not pass");
+        } catch (JsonMappingException e0) {
+            verifyException(e0, "No suitable constructor");
+            e = e0;
+        }
+        // but should be able to serialize new exception we got
+        String json = MAPPER.writeValueAsString(e);
+        JsonNode root = MAPPER.readTree(json);
+        String msg = root.path("message").asText();
+        String MATCH = "No suitable constructor";
+        if (!msg.contains(MATCH)) {
+            fail("Exception should contain '"+MATCH+"', does not: '"+msg+"'");
+        }
+    }
 }