diff --git a/modules/src/main/java/org/archive/modules/fetcher/FetchHTTP2.java b/modules/src/main/java/org/archive/modules/fetcher/FetchHTTP2.java index 74dea299..8e133633 100644 --- a/modules/src/main/java/org/archive/modules/fetcher/FetchHTTP2.java +++ b/modules/src/main/java/org/archive/modules/fetcher/FetchHTTP2.java @@ -822,6 +822,9 @@ public class FetchHTTP2 extends AbstractFetchHTTP implements Lifecycle { protected RecordingHttpClient getHttpClient() { if (httpClient == null) { httpClient = new RecordingHttpClient(getServerCache()); + + // XXX should this in the constructor? in configureRequest()? somewhere else? + HttpClientParams.setRedirecting(httpClient.getParams(), false); } return httpClient; diff --git a/modules/src/main/java/org/archive/modules/fetcher/RecordingHttpClient.java b/modules/src/main/java/org/archive/modules/fetcher/RecordingHttpClient.java index 049c8d4f..33aaf14a 100644 --- a/modules/src/main/java/org/archive/modules/fetcher/RecordingHttpClient.java +++ b/modules/src/main/java/org/archive/modules/fetcher/RecordingHttpClient.java @@ -64,13 +64,13 @@ public class RecordingHttpClient extends DefaultHttpClient { this.setServerCache(serverCache); // XXX uhh? see HeritrixHttpMethodRetryHandler ?? - setHttpRequestRetryHandler(new HttpRequestRetryHandler() { - @Override - public boolean retryRequest(IOException exception, int executionCount, - HttpContext context) { - return false; - } - }); +// setHttpRequestRetryHandler(new HttpRequestRetryHandler() { +// @Override +// public boolean retryRequest(IOException exception, int executionCount, +// HttpContext context) { +// return false; +// } +// }); // never reuse (no keep-alive) setReuseStrategy(new ConnectionReuseStrategy() {