Merge pull request yasserg#314 from valfirst/use-pre-defined-http-hea…

…der-constants Use pre-defined HTTP header constants
shiguang1120 · Apr 3, 2018 · 3cb4c82 · 3cb4c82
2 parents ab2823c + 0e93a57
commit 3cb4c82
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/crawler4j/src/main/java/edu/uci/ics/crawler4j/fetcher/PageFetcher.java b/crawler4j/src/main/java/edu/uci/ics/crawler4j/fetcher/PageFetcher.java
@@ -30,6 +30,7 @@
 import javax.net.ssl.SSLContext;
 
 import org.apache.http.Header;
+import org.apache.http.HttpHeaders;
 import org.apache.http.HttpHost;
 import org.apache.http.HttpStatus;
 import org.apache.http.NameValuePair;
@@ -277,7 +278,7 @@ public PageFetchResult fetchPage(WebURL webUrl)
                     statusCode == 308) { // todo follow
                 // https://issues.apache.org/jira/browse/HTTPCORE-389
 
-                Header header = response.getFirstHeader("Location");
+                Header header = response.getFirstHeader(HttpHeaders.LOCATION);
                 if (header != null) {
                     String movedToUrl =
                             URLCanonicalizer.getCanonicalURL(header.getValue(), toFetchURL);
@@ -296,7 +297,7 @@ public PageFetchResult fetchPage(WebURL webUrl)
                 if (fetchResult.getEntity() != null) {
                     long size = fetchResult.getEntity().getContentLength();
                     if (size == -1) {
-                        Header length = response.getLastHeader("Content-Length");
+                        Header length = response.getLastHeader(HttpHeaders.CONTENT_LENGTH);
                         if (length == null) {
                             length = response.getLastHeader("Content-length");
                         }