Make sure AbstractHttpExceptions have a meaningful toString().

This commit is contained in:
Andreas Schildbach 2015-10-23 21:21:14 +02:00
parent 400b06cb8d
commit 9c853ad5d6
2 changed files with 12 additions and 8 deletions

View file

@ -32,18 +32,16 @@ public abstract class AbstractHttpException extends IOException
private final URL url; private final URL url;
private final Reader errorReader; private final Reader errorReader;
public AbstractHttpException(final URL url, final Reader errorReader) public AbstractHttpException(final URL url)
{ {
super(); this(url, null);
this.url = url;
this.errorReader = errorReader;
} }
public AbstractHttpException(final URL url, final String message) public AbstractHttpException(final URL url, final Reader errorReader)
{ {
super(message); super(url.toString());
this.url = url; this.url = url;
this.errorReader = null; this.errorReader = errorReader;
} }
public URL getUrl() public URL getUrl()

View file

@ -29,7 +29,7 @@ public class UnexpectedRedirectException extends AbstractHttpException
public UnexpectedRedirectException(final URL originalUrl, final URL redirectedUrl) public UnexpectedRedirectException(final URL originalUrl, final URL redirectedUrl)
{ {
super(originalUrl, originalUrl + " -> " + redirectedUrl); super(originalUrl);
this.redirectedUrl = redirectedUrl; this.redirectedUrl = redirectedUrl;
} }
@ -37,4 +37,10 @@ public class UnexpectedRedirectException extends AbstractHttpException
{ {
return redirectedUrl; return redirectedUrl;
} }
@Override
public String toString()
{
return getClass().getName() + ": " + getUrl() + " -> " + redirectedUrl;
}
} }