You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by mi...@apache.org on 2019/12/05 20:23:14 UTC

[httpcomponents-client] 08/08: Return null on async retry

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch HTTPCLIENT-2034
in repository https://gitbox.apache.org/repos/asf/httpcomponents-client.git

commit 0d6419ff2de66b83c8591b1eac3a3c1d0e2646b5
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Thu Dec 5 21:21:04 2019 +0100

    Return null on async retry
---
 .../apache/hc/client5/http/impl/async/AsyncHttpRequestRetryExec.java   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/httpclient5/src/main/java/org/apache/hc/client5/http/impl/async/AsyncHttpRequestRetryExec.java b/httpclient5/src/main/java/org/apache/hc/client5/http/impl/async/AsyncHttpRequestRetryExec.java
index 01bd062..475ba65 100644
--- a/httpclient5/src/main/java/org/apache/hc/client5/http/impl/async/AsyncHttpRequestRetryExec.java
+++ b/httpclient5/src/main/java/org/apache/hc/client5/http/impl/async/AsyncHttpRequestRetryExec.java
@@ -125,6 +125,7 @@ public final class AsyncHttpRequestRetryExec implements AsyncExecChainHandler {
                             }
                         }
                         internalExecute(execCount + 1, request, entityProducer, scope, chain, asyncExecCallback);
+                        return null;
                     } else {
                         return asyncExecCallback.handleResponse(response, entityDetails);
                     }
@@ -133,8 +134,6 @@ public final class AsyncHttpRequestRetryExec implements AsyncExecChainHandler {
                     //response.close();
                     throw ex;
                 }
-
-                return asyncExecCallback.handleResponse(response, entityDetails);
             }
 
             @Override