summaryrefslogtreecommitdiffstats
path: root/src/org/apache/http/impl/client/DefaultRequestDirector.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/org/apache/http/impl/client/DefaultRequestDirector.java')
-rw-r--r--src/org/apache/http/impl/client/DefaultRequestDirector.java26
1 files changed, 23 insertions, 3 deletions
diff --git a/src/org/apache/http/impl/client/DefaultRequestDirector.java b/src/org/apache/http/impl/client/DefaultRequestDirector.java
index a95c522..bfdddd6 100644
--- a/src/org/apache/http/impl/client/DefaultRequestDirector.java
+++ b/src/org/apache/http/impl/client/DefaultRequestDirector.java
@@ -67,6 +67,7 @@ import org.apache.http.client.RedirectHandler;
import org.apache.http.client.UserTokenHandler;
import org.apache.http.client.methods.AbortableHttpRequest;
import org.apache.http.client.methods.HttpGet;
+import org.apache.http.client.methods.HttpUriRequest;
import org.apache.http.client.params.ClientPNames;
import org.apache.http.client.params.HttpClientParams;
import org.apache.http.client.protocol.ClientContext;
@@ -357,7 +358,13 @@ public class DefaultRequestDirector implements RequestDirector {
// Reopen connection if needed
if (!managedConn.isOpen()) {
managedConn.open(route, context, params);
- }
+ }
+ // BEGIN android-added
+ else {
+ // b/3241899 set the per request timeout parameter on reused connections
+ managedConn.setSocketTimeout(HttpConnectionParams.getSoTimeout(params));
+ }
+ // END android-added
try {
establishRoute(route, context);
@@ -569,8 +576,21 @@ public class DefaultRequestDirector implements RequestDirector {
ClientPNames.DEFAULT_HOST);
}
if (target == null) {
- throw new IllegalStateException
- ("Target host must not be null, or set in parameters.");
+ // BEGIN android-changed
+ // If the URI was malformed, make it obvious where there's no host component
+ String scheme = null;
+ String host = null;
+ String path = null;
+ URI uri;
+ if (request instanceof HttpUriRequest
+ && (uri = ((HttpUriRequest) request).getURI()) != null) {
+ scheme = uri.getScheme();
+ host = uri.getHost();
+ path = uri.getPath();
+ }
+ throw new IllegalStateException( "Target host must not be null, or set in parameters."
+ + " scheme=" + scheme + ", host=" + host + ", path=" + path);
+ // END android-changed
}
return this.routePlanner.determineRoute(target, request, context);