issue #451 pass original failure reason to new mapped response in ifF…

…ailure

CHANGELOG.md

@@ -1,5 +1,7 @@
 ## 3.14.0
 * issue #461 return HttpResponse<Empty> for asEmpty()
+* issue #450 add authenticated proxies to request
+* issue #451 pass original failure reason to new mapped response in ifFailure
 
 ## 3.13.13
 * Cookie dates always follow US Locale to avoid invalid unicode in headers

unirest-bdd-tests/src/test/java/BehaviorTests/ErrorParsingTest.java

@@ -30,6 +30,7 @@
 import com.fasterxml.jackson.databind.DeserializationFeature;
 import kong.unirest.HttpResponse;
 import kong.unirest.Unirest;
+import org.eclipse.jetty.server.Request;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 
@@ -39,6 +40,22 @@
 class ErrorParsingTest extends BddTest {
     private boolean errorCalled;
 
+    @Test
+    void passParsingErrorsOnInFalure() {
+        MockServer.setStringResponse("not json");
+
+        Unirest.get(MockServer.GET)
+                .asObject(RequestCapture.class)
+                .ifFailure(String.class, r ->
+                        {
+                            assertTrue(r.getParsingError().isPresent());
+                            assertTrue(r.getParsingError().get().getMessage().contains("jackson"));
+                            assertEquals("not json", r.getParsingError().get().getOriginalBody());
+                        }
+                )
+                .ifSuccess(s -> {throw new RuntimeException("no");});
+    }
+
     @Test
     void parsingAnAlternativeErrorObject() {
         MockServer.setJsonAsResponse(new ErrorThing("boom!"));

unirest/src/main/java/kong/unirest/BaseResponse.java

@@ -155,7 +155,9 @@ private String getErrorBody() {
     public <E> HttpResponse<T> ifFailure(Class<? extends E> errorClass, Consumer<HttpResponse<E>> consumer) {
         if (!isSuccess()) {
             E error = mapError(errorClass);
-            consumer.accept(new BasicResponse(this, error));
+            BasicResponse br = new BasicResponse(this, error);
+            getParsingError().ifPresent(p -> br.setParsingException(p.getOriginalBody(), p));
+            consumer.accept(br);
         }
         return this;
     }