From 29b3b92b85524169622e80a8c8aab697ed94e779 Mon Sep 17 00:00:00 2001 From: Athou Date: Tue, 23 Apr 2013 09:14:18 +0200 Subject: [PATCH] throw an exception when needed, prevents html error pages parsing --- src/main/java/com/commafeed/backend/HttpGetter.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/main/java/com/commafeed/backend/HttpGetter.java b/src/main/java/com/commafeed/backend/HttpGetter.java index 08c221d9..68cb9cea 100644 --- a/src/main/java/com/commafeed/backend/HttpGetter.java +++ b/src/main/java/com/commafeed/backend/HttpGetter.java @@ -97,12 +97,19 @@ public class HttpGetter { HttpResponse response = null; try { response = client.execute(httpget); - if (response.getStatusLine().getStatusCode() == HttpStatus.SC_NOT_MODIFIED) { + int code = response.getStatusLine().getStatusCode(); + if (code == HttpStatus.SC_NOT_MODIFIED) { throw new NotModifiedException(); + } else if (code >= 300) { + throw new HttpResponseException(code, + "Server returned HTTP error code " + code); } + } catch (HttpResponseException e) { if (e.getStatusCode() == HttpStatus.SC_NOT_MODIFIED) { throw new NotModifiedException(); + } else { + throw e; } } Header lastModifiedHeader = response