From b79f0550a22308e6aab1522e9bd323c68ffe1df3 Mon Sep 17 00:00:00 2001 From: Archimedes Trajano Date: Thu, 24 Sep 2015 00:35:35 -0400 Subject: [PATCH] Store problematic JSON in exception Rather than logging the JSON that is problematic store it as part of the exception. Remove the logger since it is no longer being used. --- .../jsonpath/spi/json/JacksonJsonNodeJsonProvider.java | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/json-path/src/main/java/com/jayway/jsonpath/spi/json/JacksonJsonNodeJsonProvider.java b/json-path/src/main/java/com/jayway/jsonpath/spi/json/JacksonJsonNodeJsonProvider.java index 8ef4cd18..a7d58255 100644 --- a/json-path/src/main/java/com/jayway/jsonpath/spi/json/JacksonJsonNodeJsonProvider.java +++ b/json-path/src/main/java/com/jayway/jsonpath/spi/json/JacksonJsonNodeJsonProvider.java @@ -8,8 +8,6 @@ import com.fasterxml.jackson.databind.node.ObjectNode; import com.fasterxml.jackson.databind.node.TextNode; import com.jayway.jsonpath.InvalidJsonException; import com.jayway.jsonpath.JsonPathException; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; import java.io.IOException; import java.io.InputStream; @@ -21,8 +19,6 @@ import java.util.List; public class JacksonJsonNodeJsonProvider extends AbstractJsonProvider { - private static final Logger logger = LoggerFactory.getLogger(JacksonJsonProvider.class); - private static final ObjectMapper defaultObjectMapper = new ObjectMapper(); protected ObjectMapper objectMapper; @@ -52,8 +48,7 @@ public class JacksonJsonNodeJsonProvider extends AbstractJsonProvider { try { return objectMapper.readTree(json); } catch (IOException e) { - logger.debug("Invalid JSON: \n" + json); - throw new InvalidJsonException(e); + throw new InvalidJsonException(e, json); } }