Browse Source

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.
pull/126/head
Archimedes Trajano 9 years ago
parent
commit
b79f0550a2
  1. 7
      json-path/src/main/java/com/jayway/jsonpath/spi/json/JacksonJsonNodeJsonProvider.java

7
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.fasterxml.jackson.databind.node.TextNode;
import com.jayway.jsonpath.InvalidJsonException; import com.jayway.jsonpath.InvalidJsonException;
import com.jayway.jsonpath.JsonPathException; import com.jayway.jsonpath.JsonPathException;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
@ -21,8 +19,6 @@ import java.util.List;
public class JacksonJsonNodeJsonProvider extends AbstractJsonProvider { public class JacksonJsonNodeJsonProvider extends AbstractJsonProvider {
private static final Logger logger = LoggerFactory.getLogger(JacksonJsonProvider.class);
private static final ObjectMapper defaultObjectMapper = new ObjectMapper(); private static final ObjectMapper defaultObjectMapper = new ObjectMapper();
protected ObjectMapper objectMapper; protected ObjectMapper objectMapper;
@ -52,8 +48,7 @@ public class JacksonJsonNodeJsonProvider extends AbstractJsonProvider {
try { try {
return objectMapper.readTree(json); return objectMapper.readTree(json);
} catch (IOException e) { } catch (IOException e) {
logger.debug("Invalid JSON: \n" + json); throw new InvalidJsonException(e, json);
throw new InvalidJsonException(e);
} }
} }

Loading…
Cancel
Save