Skip to content

Commit 0dbfaf7

Browse files
committed
Remove useless log.
1 parent bda5153 commit 0dbfaf7

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

webmagic-core/src/main/java/us/codecraft/webmagic/downloader/HttpClientDownloader.java

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@
1212
import org.apache.http.client.methods.CloseableHttpResponse;
1313
import org.apache.http.impl.client.CloseableHttpClient;
1414
import org.apache.http.util.EntityUtils;
15-
import org.slf4j.Logger;
16-
import org.slf4j.LoggerFactory;
1715

1816
import us.codecraft.webmagic.Page;
1917
import us.codecraft.webmagic.Request;
@@ -33,8 +31,6 @@
3331
*/
3432
public class HttpClientDownloader extends AbstractDownloader {
3533

36-
private Logger logger = LoggerFactory.getLogger(getClass());
37-
3834
private final Map<String, CloseableHttpClient> httpClients = new HashMap<String, CloseableHttpClient>();
3935

4036
private HttpClientGenerator httpClientGenerator = new HttpClientGenerator();
@@ -84,16 +80,10 @@ public Page download(Request request, Task task) {
8480
try {
8581
httpResponse = httpClient.execute(requestContext.getHttpUriRequest(), requestContext.getHttpClientContext());
8682
page = handleResponse(request, request.getCharset() != null ? request.getCharset() : task.getSite().getCharset(), httpResponse, task);
87-
8883
onSuccess(page, task);
89-
logger.info("Download page success: {}", request.getUrl());
90-
9184
return page;
9285
} catch (IOException e) {
93-
9486
onError(page, task, e);
95-
logger.info("Download page error: {}", request.getUrl(), e);
96-
9787
return page;
9888
} finally {
9989
if (httpResponse != null) {
@@ -138,7 +128,6 @@ private String getHtmlCharset(String contentType, byte[] contentBytes, Task task
138128
String charset = CharsetUtils.detectCharset(contentType, contentBytes);
139129
if (charset == null) {
140130
charset = Optional.ofNullable(task.getSite().getDefaultCharset()).orElseGet(Charset.defaultCharset()::name);
141-
logger.info("Charset autodetect failed, use {} as charset.", task.getSite().getDefaultCharset());
142131
}
143132
return charset;
144133
}

0 commit comments

Comments
 (0)