Jarek-Demo
diff --git a/‎src/main/java/org/kohsuke/github/GHApiInfo.java
Copy file name to clipboardExpand all lines: src/main/java/org/kohsuke/github/GHApiInfo.java
-14Lines changed: 0 additions & 14 deletions b/‎src/main/java/org/kohsuke/github/GHApiInfo.java
Copy file name to clipboardExpand all lines: src/main/java/org/kohsuke/github/GHApiInfo.java
-14Lines changed: 0 additions & 14 deletions
diff --git a/‎src/main/java/org/kohsuke/github/GitHub.java
Copy file name to clipboardExpand all lines: src/main/java/org/kohsuke/github/GitHub.java
+18-12Lines changed: 18 additions & 12 deletions b/‎src/main/java/org/kohsuke/github/GitHub.java
Copy file name to clipboardExpand all lines: src/main/java/org/kohsuke/github/GitHub.java
+18-12Lines changed: 18 additions & 12 deletions
diff --git a/‎src/test/java/org/kohsuke/github/GitHubTest.java
Copy file name to clipboardExpand all lines: src/test/java/org/kohsuke/github/GitHubTest.java
+1-1Lines changed: 1 addition & 1 deletion b/‎src/test/java/org/kohsuke/github/GitHubTest.java
Copy file name to clipboardExpand all lines: src/test/java/org/kohsuke/github/GitHubTest.java
+1-1Lines changed: 1 addition & 1 deletion
@@ -455,21 +455,27 @@ public boolean isCredentialValid() throws IOException {
         }
     }
 
+    private static class GHApiInfo {
+        private String rate_limit_url;
+
+        void check(String apiUrl) throws IOException {
+            if (rate_limit_url==null)
+                throw new IOException(apiUrl+" doesn't look like GitHub API URL");
+
+            // make sure that the URL is legitimate
+            new URL(rate_limit_url);
+        }
+    }
+
     /**
      * Ensures that the API URL is valid.
+     *
+     * <p>
+     * This method returns normally if the endpoint is reachable and verified to be GitHub API URL.
+     * Otherwise this method throws {@link IOException} to indicate the problem.
      */
-    public boolean isApiUrlValid() throws IOException {
-        try {
-            GHApiInfo apiInfo = retrieve().to("/", GHApiInfo.class);
-            try {
-                new URL(apiInfo.getRateLimitUrl());
-                return true;
-            } catch (MalformedURLException mue) {
-                return false;
-            }
-        } catch (IOException e) {
-            return false;
-        }
+    public void checkApiUrlValidity() throws IOException {
+        retrieve().to("/", GHApiInfo.class).check(apiUrl);
     }
 
     /**
 
@@ -124,6 +124,6 @@ public void testGitHubEnterpriseDoesNotHaveRateLimit() throws IOException {
     @Test
     public void testGitHubIsApiUrlValid() throws IOException {
         GitHub github = GitHub.connectAnonymously();
-        assertTrue(github.isApiUrlValid());
+        github.checkApiUrlValidity();
     }
 }
Original file line number	Diff line number	Diff line change
`@@ -124,6 +124,6 @@ public void testGitHubEnterpriseDoesNotHaveRateLimit() throws IOException {`
`124`	`124`	`@Test`
`125`	`125`	`public void testGitHubIsApiUrlValid() throws IOException {`
`126`	`126`	`GitHub github = GitHub.connectAnonymously();`
`127`		`- assertTrue(github.isApiUrlValid());`
	`127`	`+ github.checkApiUrlValidity();`
`128`	`128`	`}`
`129`	`129`	`}`