FIX: return null in case of a 404 error instead of throwing an exception

Release version 1.6.0
SOBotics · Oct 16, 2020 · fe43636 · fe43636
1 parent c8e06fb
commit fe43636
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/checkstyle_checks.xml b/checkstyle_checks.xml
@@ -426,7 +426,6 @@
       <property name="format" value="^([^\s\/*])"/>
       <message key="todo.match" value="Comment text should start with space."/>
     </module>
-    <module name="TrailingComment"/>
     <module name="UncommentedMain">
       <property name="excludedClasses" value="\.(Application|JavadocPropertiesGenerator)$"/>
     </module>

diff --git a/pom.xml b/pom.xml
@@ -6,7 +6,7 @@
 
     <groupId>bugs.stackoverflow</groupId>
     <artifactId>belisarius</artifactId>
-    <version>1.5.11</version>
+    <version>1.6.0</version>
 
     <properties>
         <project.java.version>1.8</project.java.version>

diff --git a/src/main/java/bugs/stackoverflow/belisarius/utils/JsonUtils.java b/src/main/java/bugs/stackoverflow/belisarius/utils/JsonUtils.java
@@ -65,7 +65,9 @@ public static String getHtml(String url) throws IOException {
         Connection.Response response = Jsoup.connect(url).method(Connection.Method.GET)
                                             .ignoreContentType(true).ignoreHttpErrors(true).execute();
         String body = response.body();
-        if (response.statusCode() != 200) {
+        if (response.statusCode() == 404) { // revision did not affect the body
+            return null;
+        } else if (response.statusCode() != 200) {
             throw new IOException("HTTP " + response.statusCode() + " error fetching " + url + ".");
         }
         return Jsoup.parse(body).getElementsByTag("pre").text();