summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJesse Wilson <jessewilson@google.com>2011-01-06 18:23:53 -0800
committerAndroid Git Automerger <android-git-automerger@android.com>2011-01-06 18:23:53 -0800
commit9a8b5e2e45cc413b398b76481ce42c748ba6dc94 (patch)
tree0ee4fa60286ff5efd27281649ad6f81a796de593
parent741ca79db8c9076ba1c9f2a47cfa839da9e3f53f (diff)
parent2d8fd9bea30efcae4cc418e223f2c61febe0caa0 (diff)
downloadexternal_apache-http-9a8b5e2e45cc413b398b76481ce42c748ba6dc94.zip
external_apache-http-9a8b5e2e45cc413b398b76481ce42c748ba6dc94.tar.gz
external_apache-http-9a8b5e2e45cc413b398b76481ce42c748ba6dc94.tar.bz2
am 2d8fd9be: Help the caller along if a malformed URI is used to make an HTTP request.
* commit '2d8fd9bea30efcae4cc418e223f2c61febe0caa0': Help the caller along if a malformed URI is used to make an HTTP request.
-rw-r--r--src/org/apache/http/impl/client/DefaultRequestDirector.java18
1 files changed, 16 insertions, 2 deletions
diff --git a/src/org/apache/http/impl/client/DefaultRequestDirector.java b/src/org/apache/http/impl/client/DefaultRequestDirector.java
index b8f380b..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;
@@ -575,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);