Downloader: Check content-type for null
This commit is contained in:
parent
926f3912cf
commit
95f1beb479
|
@ -185,8 +185,6 @@ public class HttpDownloader extends Downloader {
|
||||||
// fail with a file type error when the content type is text and
|
// fail with a file type error when the content type is text and
|
||||||
// the reported content length is less than 100kb (or no length is given)
|
// the reported content length is less than 100kb (or no length is given)
|
||||||
if(request.getFeedfileType() == FeedMedia.FEEDFILETYPE_FEEDMEDIA) {
|
if(request.getFeedfileType() == FeedMedia.FEEDFILETYPE_FEEDMEDIA) {
|
||||||
String contentType = response.header("Content-Type");
|
|
||||||
Log.d(TAG, "content type: " + contentType);
|
|
||||||
int contentLength = -1;
|
int contentLength = -1;
|
||||||
String contentLen = response.header("Content-Length");
|
String contentLen = response.header("Content-Length");
|
||||||
if(contentLen != null) {
|
if(contentLen != null) {
|
||||||
|
@ -195,7 +193,10 @@ public class HttpDownloader extends Downloader {
|
||||||
} catch(NumberFormatException e) {}
|
} catch(NumberFormatException e) {}
|
||||||
}
|
}
|
||||||
Log.d(TAG, "content length: " + contentLength);
|
Log.d(TAG, "content length: " + contentLength);
|
||||||
if(contentType.startsWith("text/") && contentLength < 100 * 1024) {
|
String contentType = response.header("Content-Type");
|
||||||
|
Log.d(TAG, "content type: " + contentType);
|
||||||
|
if(contentType != null && contentType.startsWith("text/") &&
|
||||||
|
contentLength < 100 * 1024) {
|
||||||
onFail(DownloadError.ERROR_FILE_TYPE, null);
|
onFail(DownloadError.ERROR_FILE_TYPE, null);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue