Fix #1794

Author: cowtowncoder

release-notes/VERSION

@@ -6,6 +6,8 @@ Project: jackson-databind
 2.9.3 (not yet released)
 
 #1604: Nested type arguments doesn't work with polymorphic types
+#1794: `StackTraceElementDeserializer` not working if field visibility changed
+ (reported by dsingley@github)
 #1799: Allow creation of custom sub-types of `NullNode`, `BooleanNode`, `MissingNode`
 #1804: `ValueInstantiator.canInstantiate()` ignores `canCreateUsingArrayDelegate()`
  (reported byb henryptung@github)

src/main/java/com/fasterxml/jackson/databind/deser/std/StackTraceElementDeserializer.java

@@ -50,9 +50,13 @@ public StackTraceElement deserialize(JsonParser p, DeserializationContext ctxt)
                     moduleName = p.getText();
                 } else if ("moduleVersion".equals(propName)) {
                     moduleVersion = p.getText();
+                } else if ("declaringClass".equals(propName)) {
+                    // 01-Nov-2017: [databind#1794] Not sure if we should but... let's prune it for now
+                    ;
                 } else {
                     handleUnknownProperty(p, ctxt, _valueClass, propName);
                 }
+                p.skipChildren(); // just in case we might get structured values
             }
             return constructValue(ctxt, className, methodName, fileName, lineNumber,
                     moduleName, moduleVersion, classLoaderName);

src/test/java/com/fasterxml/jackson/databind/exc/StackTraceElementTest.java

@@ -0,0 +1,39 @@
+package com.fasterxml.jackson.databind.exc;
+
+import com.fasterxml.jackson.annotation.*;
+import com.fasterxml.jackson.databind.*;
+
+// for [databind#1794]
+public class StackTraceElementTest extends BaseMapTest
+{
+    public static class ErrorObject {
+
+        public String throwable;
+        public String message;
+
+//        @JsonDeserialize(contentUsing = StackTraceElementDeserializer.class)
+        public StackTraceElement[] stackTrace;
+
+        ErrorObject() {}
+
+        public ErrorObject(Throwable throwable) {
+            this.throwable = throwable.getClass().getName();
+            message = throwable.getMessage();
+            stackTrace = throwable.getStackTrace();
+        }
+    }
+
+    // for [databind#1794] where extra `declaringClass` is serialized from private field.
+    public void testCustomStackTraceDeser() throws Exception
+    {
+        ObjectMapper mapper = newObjectMapper();
+        mapper.setVisibility(PropertyAccessor.FIELD, JsonAutoDetect.Visibility.ANY);
+
+        String json = mapper
+                .writerWithDefaultPrettyPrinter()
+                .writeValueAsString(new ErrorObject(new Exception("exception message")));
+
+        ErrorObject result = mapper.readValue(json, ErrorObject.class);
+        assertNotNull(result);
+    }
+}