Change credential check to use rate limit

Fixes #582

Author: bitwiseman

src/main/java/org/kohsuke/github/GitHubClient.java

@@ -148,13 +148,15 @@ private <T> T fetch(Class<T> type, String urlPath) throws IOException {
     }
 
     /**
-     * Ensures that the credential is valid.
+     * Ensures that the credential for this client is valid.
      *
      * @return the boolean
      */
     public boolean isCredentialValid() {
         try {
-            fetch(GHUser.class, "/user");
+            // If 404, ratelimit returns a default value.
+            // This works as credential test because invalid credentials returns 401, not 404
+            getRateLimit();
             return true;
         } catch (IOException e) {
             if (LOGGER.isLoggable(FINE))

src/test/java/org/kohsuke/github/AppTest.java

@@ -77,8 +77,12 @@ private void cleanupUserRepository(final String name) throws IOException {
     @Test
     public void testCredentialValid() throws IOException {
         assertTrue(gitHub.isCredentialValid());
+
+        // TODO: Doh, this doesn't go through the proxy. Needs rewriting.
         GitHub connect = GitHub.connect("totally", "bogus");
         assertFalse(connect.isCredentialValid());
+
+        // TODO: Add GHE test - where RateLimit returns 404.
     }
 
     @Test

src/test/resources/org/kohsuke/github/AppTest/wiremock/testCredentialValid/__files/rate_limit-2.json

@@ -0,0 +1,34 @@
+{
+  "resources": {
+    "core": {
+      "limit": 5000,
+      "remaining": 4998,
+      "reset": 1585671841
+    },
+    "search": {
+      "limit": 30,
+      "remaining": 30,
+      "reset": 1585668302
+    },
+    "graphql": {
+      "limit": 5000,
+      "remaining": 5000,
+      "reset": 1585671842
+    },
+    "integration_manifest": {
+      "limit": 5000,
+      "remaining": 5000,
+      "reset": 1585671842
+    },
+    "source_import": {
+      "limit": 100,
+      "remaining": 100,
+      "reset": 1585668302
+    }
+  },
+  "rate": {
+    "limit": 5000,
+    "remaining": 4998,
+    "reset": 1585671841
+  }
+}

src/test/resources/org/kohsuke/github/AppTest/wiremock/testCredentialValid/__files/user-1.json

@@ -24,14 +24,14 @@
   "email": "[email protected]",
   "hireable": null,
   "bio": "https://twitter.com/bitwiseman",
-  "public_repos": 167,
-  "public_gists": 4,
-  "followers": 136,
+  "public_repos": 181,
+  "public_gists": 7,
+  "followers": 151,
   "following": 9,
   "created_at": "2012-07-11T20:38:33Z",
-  "updated_at": "2019-09-24T19:32:29Z",
-  "private_gists": 7,
-  "total_private_repos": 9,
+  "updated_at": "2020-03-27T19:14:56Z",
+  "private_gists": 8,
+  "total_private_repos": 10,
   "owned_private_repos": 0,
   "disk_usage": 33697,
   "collaborators": 0,

src/test/resources/org/kohsuke/github/AppTest/wiremock/testCredentialValid/mappings/rate_limit-2.json

@@ -0,0 +1,41 @@
+{
+  "id": "776ae4cc-bd8c-4c7d-b299-4ba1b2c11ad0",
+  "name": "rate_limit",
+  "request": {
+    "url": "/rate_limit",
+    "method": "GET",
+    "headers": {
+      "Accept": {
+        "equalTo": "text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2"
+      }
+    }
+  },
+  "response": {
+    "status": 200,
+    "bodyFileName": "rate_limit-2.json",
+    "headers": {
+      "Date": "Tue, 31 Mar 2020 15:24:02 GMT",
+      "Content-Type": "application/json; charset=utf-8",
+      "Server": "GitHub.com",
+      "Status": "200 OK",
+      "X-RateLimit-Limit": "5000",
+      "X-RateLimit-Remaining": "4998",
+      "X-RateLimit-Reset": "1585671841",
+      "Cache-Control": "no-cache",
+      "X-OAuth-Scopes": "admin:org, admin:org_hook, admin:public_key, admin:repo_hook, delete_repo, gist, notifications, repo, user, write:discussion",
+      "X-Accepted-OAuth-Scopes": "",
+      "X-GitHub-Media-Type": "unknown, github.v3",
+      "Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
+      "X-Frame-Options": "deny",
+      "X-Content-Type-Options": "nosniff",
+      "X-XSS-Protection": "1; mode=block",
+      "Referrer-Policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
+      "Content-Security-Policy": "default-src 'none'",
+      "Vary": "Accept-Encoding, Accept, X-Requested-With",
+      "X-GitHub-Request-Id": "FEBE:52B1:FCCEE:123132:5E836091"
+    }
+  },
+  "uuid": "776ae4cc-bd8c-4c7d-b299-4ba1b2c11ad0",
+  "persistent": true,
+  "insertionIndex": 2
+}

src/test/resources/org/kohsuke/github/AppTest/wiremock/testCredentialValid/mappings/user-1.json

@@ -1,5 +1,5 @@
 {
-  "id": "eba96b24-3354-4d21-bd2a-388a13ba9832",
+  "id": "218a25c5-b10d-4371-a446-bd4b220c2402",
   "name": "user",
   "request": {
     "url": "/user",
@@ -14,35 +14,33 @@
     "status": 200,
     "bodyFileName": "user-1.json",
     "headers": {
-      "Date": "Wed, 02 Oct 2019 21:39:57 GMT",
+      "Date": "Tue, 31 Mar 2020 15:24:01 GMT",
       "Content-Type": "application/json; charset=utf-8",
       "Server": "GitHub.com",
       "Status": "200 OK",
       "X-RateLimit-Limit": "5000",
-      "X-RateLimit-Remaining": "4964",
-      "X-RateLimit-Reset": "1570055937",
+      "X-RateLimit-Remaining": "4999",
+      "X-RateLimit-Reset": "1585671841",
       "Cache-Control": "private, max-age=60, s-maxage=60",
       "Vary": [
         "Accept, Authorization, Cookie, X-GitHub-OTP",
-        "Accept-Encoding"
+        "Accept-Encoding, Accept, X-Requested-With"
       ],
-      "ETag": "W/\"cf6199fecf47b59c42190e1e11147ee2\"",
-      "Last-Modified": "Tue, 24 Sep 2019 19:32:29 GMT",
+      "ETag": "W/\"740bb7db37d5437d08ea1aa5e652cf37\"",
+      "Last-Modified": "Fri, 27 Mar 2020 19:14:56 GMT",
       "X-OAuth-Scopes": "admin:org, admin:org_hook, admin:public_key, admin:repo_hook, delete_repo, gist, notifications, repo, user, write:discussion",
       "X-Accepted-OAuth-Scopes": "",
       "X-GitHub-Media-Type": "unknown, github.v3",
-      "Access-Control-Expose-Headers": "ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type",
-      "Access-Control-Allow-Origin": "*",
       "Strict-Transport-Security": "max-age=31536000; includeSubdomains; preload",
       "X-Frame-Options": "deny",
       "X-Content-Type-Options": "nosniff",
       "X-XSS-Protection": "1; mode=block",
       "Referrer-Policy": "origin-when-cross-origin, strict-origin-when-cross-origin",
       "Content-Security-Policy": "default-src 'none'",
-      "X-GitHub-Request-Id": "C2BA:67D6:CA2979:EFEEDC:5D95192D"
+      "X-GitHub-Request-Id": "FEBE:52B1:FCCA5:12311D:5E836091"
     }
   },
-  "uuid": "eba96b24-3354-4d21-bd2a-388a13ba9832",
+  "uuid": "218a25c5-b10d-4371-a446-bd4b220c2402",
   "persistent": true,
   "insertionIndex": 1
 }