Improve timeouts and exception handling (#18)

* Ignore eclipse.jdt.ls project files

* Add cause to thrown exceptions where possible

This helps with debugging underlying issues

* Set callTimeout to 2x the http timeouts

This is 2x the other timeouts because I don't want to change the
current http timeout behavior. This overall timeout is only a safety
measure for when the other timeouts fail us

Author: marshalium

.gitignore

@@ -1 +1,4 @@
-/target/
+target
+.classpath
+.project
+.settings

src/main/java/com/recombee/api_client/RecombeeClient.java

@@ -917,6 +917,7 @@ private String sendRequest(Request request) throws ApiException {
                 .connectTimeout(request.getTimeout(), TimeUnit.MILLISECONDS)
                 .readTimeout(request.getTimeout(), TimeUnit.MILLISECONDS)
                 .writeTimeout(request.getTimeout(), TimeUnit.MILLISECONDS)
+                .callTimeout(request.getTimeout() * 2, TimeUnit.MILLISECONDS)
                 .build();
 
 
@@ -949,7 +950,7 @@ private String sendRequest(Request request) throws ApiException {
             return response.body().string();
         }
         catch (InterruptedIOException e) {
-            throw new ApiTimeoutException(request);
+            throw new ApiTimeoutException(request, e);
         }
         catch (IOException e) {
             e.printStackTrace();
@@ -1009,7 +1010,7 @@ private void checkErrors(Response response, Request request) throws ResponseExce
             throw new ResponseException(request, response.code(), response.body().string());
         } catch (IOException e) {
             e.printStackTrace();
-            throw new ResponseException(request, response.code(), "Failed to read the error from response");
+            throw new ResponseException(request, response.code(), "Failed to read the error from response", e);
         }
     }
 }

src/main/java/com/recombee/api_client/exceptions/ApiException.java

@@ -8,4 +8,8 @@ public class ApiException extends Exception {
     public ApiException(String message) {
         super(message);
     }
+
+    public ApiException(String message, Throwable cause) {
+        super(message, cause);
+    }
 }

src/main/java/com/recombee/api_client/exceptions/ApiTimeoutException.java

@@ -5,15 +5,15 @@
 /**
  * Exception thrown when a request is not processed within the timeout
  */
-public class ApiTimeoutException extends  ApiException {
+public class ApiTimeoutException extends ApiException {
 
     /**
      * Request that timed out
      */
     Request request;
 
-    public ApiTimeoutException(Request request) {
-        super(String.format("ApiTimeout: client did not get response within %s ms" , request.getTimeout()));
+    public ApiTimeoutException(Request request, Throwable cause) {
+        super(String.format("ApiTimeout: client did not get response within %s ms" , request.getTimeout()), cause);
         this.request = request;
     }
 

src/main/java/com/recombee/api_client/exceptions/ResponseException.java

@@ -5,7 +5,7 @@
 /**
  * Exception thrown when a request did not succeed (did not return 200 or 201)
  */
-public class ResponseException extends  ApiException {
+public class ResponseException extends ApiException {
 
     /**
      * Request which failed
@@ -17,7 +17,11 @@ public class ResponseException extends  ApiException {
     int statusCode;
 
     public ResponseException(Request request, int statusCode, String message) {
-        super(message);
+        this(request, statusCode, message, null);
+    }
+
+    public ResponseException(Request request, int statusCode, String message, Throwable cause) {
+        super(message, cause);
         this.request = request;
         this.statusCode = statusCode;
     }