From d71addb8a72a9836d741a827eb2011b53529ad4e Mon Sep 17 00:00:00 2001 From: cyian-1756 Date: Tue, 31 Jul 2018 16:55:21 -0400 Subject: [PATCH] improved logging message and added comment for timeouts --- .../java/com/rarchives/ripme/ripper/DownloadFileThread.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/main/java/com/rarchives/ripme/ripper/DownloadFileThread.java b/src/main/java/com/rarchives/ripme/ripper/DownloadFileThread.java index e704740e..f481919a 100644 --- a/src/main/java/com/rarchives/ripme/ripper/DownloadFileThread.java +++ b/src/main/java/com/rarchives/ripme/ripper/DownloadFileThread.java @@ -110,6 +110,8 @@ class DownloadFileThread extends Thread { huc = (HttpURLConnection) urlToDownload.openConnection(); } huc.setInstanceFollowRedirects(true); + // It is important to set both ConnectTimeout and ReadTimeout. If you don't then ripme will wait forever + // for the server to send data after connecting. huc.setConnectTimeout(TIMEOUT); huc.setReadTimeout(TIMEOUT); huc.setRequestProperty("accept", "*/*"); @@ -225,7 +227,9 @@ class DownloadFileThread extends Thread { fos.close(); break; // Download successful: break out of infinite loop } catch (SocketTimeoutException timeoutEx) { - logger.error(url.toExternalForm() + " timedout!"); + // Handle the timeout + logger.error("[!] " + url.toExternalForm() + " timedout!"); + // Download failed, break out of loop break; } catch (HttpStatusException hse) { logger.debug("HTTP status exception", hse);