]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-17827 Update usage of GitHub.com APIs to match the new versioning system.
authorAntoine Vinot <antoine.vinot@sonarsource.com>
Fri, 30 Dec 2022 10:30:35 +0000 (11:30 +0100)
committersonartech <sonartech@sonarsource.com>
Fri, 30 Dec 2022 20:02:50 +0000 (20:02 +0000)
server/sonar-alm-client/src/main/java/org/sonar/alm/client/github/GithubApplicationHttpClientImpl.java
server/sonar-alm-client/src/test/java/org/sonar/alm/client/github/GithubApplicationHttpClientImplTest.java

index 118453091428d1e9ba75e28f0b53378dc7ccf784..2d8721ce52f4986f57819a7cafe5cd9dc3c5e04b 100644 (file)
@@ -53,9 +53,8 @@ public class GithubApplicationHttpClientImpl implements GithubApplicationHttpCli
 
   private static final Logger LOG = Loggers.get(GithubApplicationHttpClientImpl.class);
   private static final Pattern NEXT_LINK_PATTERN = Pattern.compile("<([^<]+)>; rel=\"next\"");
-  private static final String GITHUB_API_VERSION_JSON = "application/vnd.github.v3+json";
-  private static final String ANTIOPE_PREVIEW_JSON = "application/vnd.github.antiope-preview+json";
-  private static final String MACHINE_MAN_PREVIEW_JSON = "application/vnd.github.machine-man-preview+json";
+  private static final String GH_API_VERSION_HEADER = "X-GitHub-Api-Version";
+  private static final String GH_API_VERSION = "2022-11-28";
 
   private final OkHttpClient client;
 
@@ -181,7 +180,7 @@ public class GithubApplicationHttpClientImpl implements GithubApplicationHttpCli
     Request.Builder url = new Request.Builder().url(toAbsoluteEndPoint(appUrl, endPoint));
     if (token != null) {
       url.addHeader("Authorization", token.getAuthorizationHeaderPrefix() + " " + token);
-      url.addHeader("Accept", String.format("%s, %s, %s", ANTIOPE_PREVIEW_JSON, MACHINE_MAN_PREVIEW_JSON, GITHUB_API_VERSION_JSON));
+      url.addHeader(GH_API_VERSION_HEADER, GH_API_VERSION);
     }
     return url;
   }
index 5b41d67b2e6eb1a5033135680a96560ed2c63a47..10fab6083c4f57b409ceeb5e350f755a98d2547c 100644 (file)
@@ -49,9 +49,9 @@ import static org.junit.Assert.fail;
 
 @RunWith(DataProviderRunner.class)
 public class GithubApplicationHttpClientImplTest {
-  private static final String BETA_API_HEADER = "application/vnd.github.antiope-preview+json, " +
-    "application/vnd.github.machine-man-preview+json, " +
-    "application/vnd.github.v3+json";
+  private static final String GH_API_VERSION_HEADER = "X-GitHub-Api-Version";
+  private static final String GH_API_VERSION = "2022-11-28";
+
   @Rule
   public MockWebServer server = new MockWebServer();
 
@@ -126,7 +126,7 @@ public class GithubApplicationHttpClientImplTest {
     assertThat(recordedRequest.getMethod()).isEqualTo("GET");
     assertThat(recordedRequest.getPath()).isEqualTo(randomEndPoint);
     assertThat(recordedRequest.getHeader("Authorization")).isEqualTo("token " + accessToken.getValue());
-    assertThat(recordedRequest.getHeader("Accept")).isEqualTo(BETA_API_HEADER);
+    assertThat(recordedRequest.getHeader(GH_API_VERSION_HEADER)).isEqualTo(GH_API_VERSION);
   }
 
   @Test
@@ -260,7 +260,7 @@ public class GithubApplicationHttpClientImplTest {
     assertThat(recordedRequest.getMethod()).isEqualTo("POST");
     assertThat(recordedRequest.getPath()).isEqualTo(randomEndPoint);
     assertThat(recordedRequest.getHeader("Authorization")).isEqualTo("token " + accessToken.getValue());
-    assertThat(recordedRequest.getHeader("Accept")).isEqualTo(BETA_API_HEADER);
+    assertThat(recordedRequest.getHeader(GH_API_VERSION_HEADER)).isEqualTo(GH_API_VERSION);
   }
 
   @Test