Fixed losing client response exception cause in some cases.

Change-Id: I659f869de3efa856095c3bcbb2d8a824b4c63e6d
Signed-off-by: Marek Potociar <[email protected]>

Author: Marek Potociar

core-client/src/main/java/org/glassfish/jersey/client/JerseyInvocation.java

@@ -1,7 +1,7 @@
 /*
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
  *
- * Copyright (c) 2011-2015 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2011-2016 Oracle and/or its affiliates. All rights reserved.
  *
  * The contents of this file are subject to the terms of either the GNU
  * General Public License Version 2 only ("GPL") or the Common Development
@@ -693,6 +693,7 @@ public <T> T invoke(final Class<T> responseType) throws ProcessingException, Web
         }
         final ClientRuntime runtime = request().getClientRuntime();
         final RequestScope requestScope = runtime.getRequestScope();
+        //noinspection Duplicates
         return requestScope.runInScope(new Producer<T>() {
             @Override
             public T call() throws ProcessingException {
@@ -715,6 +716,7 @@ public <T> T invoke(final GenericType<T> responseType) throws ProcessingExceptio
         }
         final ClientRuntime runtime = request().getClientRuntime();
         final RequestScope requestScope = runtime.getRequestScope();
+        //noinspection Duplicates
         return requestScope.runInScope(new Producer<T>() {
             @Override
             public T call() throws ProcessingException {
@@ -761,6 +763,7 @@ public <T> Future<T> submit(final Class<T> responseType) {
             throw new IllegalArgumentException(LocalizationMessages.RESPONSE_TYPE_IS_NULL());
         }
         final SettableFuture<T> responseFuture = SettableFuture.create();
+        //noinspection Duplicates
         request().getClientRuntime().submit(requestForCall(requestContext), new ResponseCallback() {
 
             @Override
@@ -823,6 +826,7 @@ public <T> Future<T> submit(final GenericType<T> responseType) {
             throw new IllegalArgumentException(LocalizationMessages.RESPONSE_TYPE_IS_NULL());
         }
         final SettableFuture<T> responseFuture = SettableFuture.create();
+        //noinspection Duplicates
         request().getClientRuntime().submit(requestForCall(requestContext), new ResponseCallback() {
 
             @Override
@@ -866,7 +870,9 @@ private <T> T translate(final ClientResponse response, final RequestScope scope,
             try {
                 return response.readEntity(responseType);
             } catch (final ProcessingException ex) {
-                throw new ResponseProcessingException(new InboundJaxrsResponse(response, scope), ex.getCause());
+                throw new ResponseProcessingException(
+                        new InboundJaxrsResponse(response, scope),
+                        ex.getCause() != null ? ex.getCause() : ex);
             } catch (final WebApplicationException ex) {
                 throw new ResponseProcessingException(new InboundJaxrsResponse(response, scope), ex);
             } catch (final Exception ex) {
@@ -910,7 +916,12 @@ public <T> Future<T> submit(final GenericType<T> responseType, final InvocationC
 
             if (responseType == null) {
                 // If we don't have response use callback to obtain param types.
-                callbackParamType = ReflectionHelper.getParameterizedTypeArguments(pair)[0];
+                final Type[] typeArguments = ReflectionHelper.getParameterizedTypeArguments(pair);
+                if (typeArguments == null || typeArguments.length == 0) {
+                    callbackParamType = Object.class;
+                } else {
+                    callbackParamType = typeArguments[0];
+                }
                 callbackParamClass = ReflectionHelper.erasure(callbackParamType);
             } else {
                 callbackParamType = responseType.getType();
@@ -958,6 +969,7 @@ public void failed(final ProcessingException error) {
             request().getClientRuntime().submit(requestForCall(requestContext), responseCallback);
         } catch (final Throwable error) {
             final ProcessingException ce;
+            //noinspection ChainOfInstanceofChecks
             if (error instanceof ProcessingException) {
                 ce = (ProcessingException) error;
                 responseFuture.setException(ce);