Merge branch '2.7'

Author: cowtowncoder

release-notes/CREDITS

@@ -476,6 +476,11 @@ 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)
+
 Artur Jonkisz (ajonkisz@github)
   * Reported #960: `@JsonCreator` not working on a factory with no arguments for ae enum type
    (2.8.0)

release-notes/VERSION

@@ -93,6 +93,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

@@ -401,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()
@@ -455,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/test/java/com/fasterxml/jackson/databind/deser/TestExceptionDeserialization.java

@@ -44,7 +44,7 @@ static class MyNoArgException extends Exception
     {
         @JsonCreator MyNoArgException() { }
     }
-    
+
     /*
     /**********************************************************
     /* Tests

src/test/java/com/fasterxml/jackson/databind/ser/TestExceptionSerialization.java

@@ -26,6 +26,15 @@ public ExceptionWithIgnoral(String msg) {
         }
     }
 
+    // [databind#1368]
+    static class NoSerdeConstructor {
+        private String strVal;
+        public String getVal() { return strVal; }
+        public NoSerdeConstructor( String strVal ) {
+            this.strVal = strVal;
+        }
+    }
+
     /*
     /**********************************************************
     /* Tests
@@ -91,4 +100,25 @@ public void testIgnorals() throws Exception
         assertNotNull(output);
         assertEquals("foobar", output.getMessage());
     }
+
+    // [databind#1368]
+    public void testJsonMappingExceptionSerialization() 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+"'");
+        }
+    }
 }