Fixed AccessTokenUtils.revokePersonalAccessToken() (#336).

Author: gmessner

src/main/java/org/gitlab4j/api/utils/AccessTokenUtils.java

@@ -36,7 +36,8 @@ public final class AccessTokenUtils {
     protected static final String PERSONAL_ACCESS_TOKEN_REGEX = "name=\\\"created-personal-access-token\\\".*data-clipboard-text=\\\"([^\\\"]*)\\\".*\\/>";
     protected static final Pattern PERSONAL_ACCESS_TOKEN_PATTERN = Pattern.compile(PERSONAL_ACCESS_TOKEN_REGEX);
 
-    protected static final String REVOKE_PERSONAL_ACCESS_TOKEN_REGEX = "<td>%s<\\/td>.*<td>%s<\\/td>.*href=\\\"([^\\\"]*)\\\">Revoke";
+    protected static final String REVOKE_PERSONAL_ACCESS_TOKEN_REGEX = "href=\\\"([^\\\"]*)\\\"";
+    protected static final Pattern REVOKE_PERSONAL_ACCESS_TOKEN_PATTERN = Pattern.compile(REVOKE_PERSONAL_ACCESS_TOKEN_REGEX);
 
     protected static final String FEED_TOKEN_REGEX = "name=\\\"feed_token\\\".*value=\\\"([^\\\"]*)\\\".*\\/>";
     protected static final Pattern FEED_TOKEN_PATTERN = Pattern.compile(FEED_TOKEN_REGEX);
@@ -230,16 +231,30 @@ public static final void revokePersonalAccessToken(final String baseUrl, final S
              * Step 3: Submit the /profile/personal_access_tokens page with the info to    *
              * revoke the first matching personal access token.                            *
              *******************************************************************************/
+            int indexOfTokenName = content.indexOf("<td>" + tokenName + "</td>");
+            if (indexOfTokenName == -1) {
+                throw new GitLabApiException("personal access token not found, aborting!");
+            }
+
+            content = content.substring(indexOfTokenName);
+            int indexOfLinkEnd = content.indexOf("</a>");
+            if (indexOfTokenName == -1) {
+                throw new GitLabApiException("personal access token not found, aborting!");
+            }
+
+            content = content.substring(0, indexOfLinkEnd);
             String scopesText = "";
             if (scopes != null && scopes.size() > 0) {
                 final StringJoiner joiner = new StringJoiner(", ");
                 scopes.forEach(s -> joiner.add(s));
                 scopesText = joiner.toString();
             }
 
-            String regex = String.format(REVOKE_PERSONAL_ACCESS_TOKEN_REGEX, tokenName, scopesText);
-            Pattern pattern = Pattern.compile(regex);
-            matcher = pattern.matcher(content);
+            if (content.indexOf(scopesText) == -1) {
+                throw new GitLabApiException("personal access token not found, aborting!");
+            }
+
+            matcher = REVOKE_PERSONAL_ACCESS_TOKEN_PATTERN.matcher(content);
             if (!matcher.find()) {
                 throw new GitLabApiException("personal access token not found, aborting!");
             }

src/test/java/org/gitlab4j/api/TestAccessTokenUtils.java

@@ -72,7 +72,7 @@ public void testCreatePersonalAccessToken() throws GitLabApiException {
         String accessToken = AccessTokenUtils.createPersonalAccessToken(
                 TEST_HOST_URL, TEST_LOGIN_USERNAME, TEST_LOGIN_PASSWORD,
                 tokenName, Arrays.asList("api", "sudo"));
-        System.out.println("Created personal access token: " + accessToken);
+        System.out.format("Created '%s' personal access token: %s%n", tokenName, accessToken);
 
         assertNotNull(accessToken);
         assertFalse(accessToken.trim().isEmpty());
@@ -82,7 +82,7 @@ public void testCreatePersonalAccessToken() throws GitLabApiException {
             AccessTokenUtils.revokePersonalAccessToken(
                 TEST_HOST_URL, TEST_LOGIN_USERNAME, TEST_LOGIN_PASSWORD,
                 tokenName, Arrays.asList("api", "sudo"));
-            System.out.println("Revoked personal access token: " + accessToken);
+            System.out.format("Revoked '%s' personal access token: %s%n", tokenName, accessToken);
         } catch (Exception ignore) {}
     }
 
@@ -107,14 +107,14 @@ public void testRevokePersonalAccessToken() throws GitLabApiException {
         String accessToken = AccessTokenUtils.createPersonalAccessToken(
                 TEST_HOST_URL, TEST_LOGIN_USERNAME, TEST_LOGIN_PASSWORD,
                 tokenName, Arrays.asList("api", "sudo"));
-        System.out.println("Created personal access token: " + accessToken);
+        System.out.format("Created '%s' personal access token: %s%n", tokenName, accessToken);
         assertNotNull(accessToken);
         assertFalse(accessToken.trim().isEmpty());
 
         AccessTokenUtils.revokePersonalAccessToken(
                 TEST_HOST_URL, TEST_LOGIN_USERNAME, TEST_LOGIN_PASSWORD,
                 tokenName, Arrays.asList("api", "sudo"));
-        System.out.println("Revoked personal access token: " + accessToken);
+        System.out.format("Revoked '%s' personal access token: %s%n", tokenName, accessToken);
     }
 
     @Test