fix(platforms): explicitly request json response from coinbase API (#…

…3121)
passportxyz · Dec 10, 2024 · 28b746e · 28b746e
1 parent b18d319
commit 28b746e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/platforms/src/Coinbase/Providers/coinbase.ts b/platforms/src/Coinbase/Providers/coinbase.ts
@@ -72,7 +72,6 @@ export const requestAccessToken = async (code: string): Promise<string | undefin
     console.log("error", e);
     handleProviderAxiosError(e, "Coinbase access token", [clientSecret, code]);
   }
-  console.log("tokenRequest", tokenRequest);
 
   return tokenRequest?.data?.access_token;
 };
@@ -84,7 +83,7 @@ export const verifyCoinbaseLogin = async (code: string): Promise<string | undefi
   try {
     // Now that we have an access token fetch the user details
     userResponse = await axios.get("https://api.coinbase.com/v2/user", {
-      headers: { Authorization: `Bearer ${accessToken}` },
+      headers: { Authorization: `Bearer ${accessToken}`, Accept: "application/json" },
     });
   } catch (e) {
     handleProviderAxiosError(e, "Coinbase user info", [accessToken, code]);

diff --git a/platforms/src/Coinbase/__tests__/coinbase.test.ts b/platforms/src/Coinbase/__tests__/coinbase.test.ts
@@ -197,7 +197,7 @@ describe("Attempt verification", function () {
     expect(mockedAxios.get).toBeCalledTimes(1);
     // Check the request to get the user
     expect(mockedAxios.get).toBeCalledWith("https://api.coinbase.com/v2/user", {
-      headers: { Authorization: "Bearer cnbstkn294745627362562" },
+      headers: { Authorization: "Bearer cnbstkn294745627362562", Accept: "application/json" },
     });
   });
 
@@ -236,7 +236,7 @@ describe("Attempt verification", function () {
     expect(mockedAxios.get).toBeCalledTimes(1);
     // Check the request to get the user
     expect(mockedAxios.get).toBeCalledWith("https://api.coinbase.com/v2/user", {
-      headers: { Authorization: "Bearer cnbstkn294745627362562" },
+      headers: { Authorization: "Bearer cnbstkn294745627362562", Accept: "application/json" },
     });
   });
 
@@ -267,7 +267,7 @@ describe("Attempt verification", function () {
     expect(mockedAxios.get).toBeCalledTimes(1);
     // Check the request to get the user
     expect(mockedAxios.get).toBeCalledWith("https://api.coinbase.com/v2/user", {
-      headers: { Authorization: "Bearer cnbstkn294745627362562" },
+      headers: { Authorization: "Bearer cnbstkn294745627362562", Accept: "application/json" },
     });
 
     expect(coinbasePayload).toMatchObject({
@@ -297,7 +297,7 @@ describe("Attempt verification", function () {
     expect(mockedAxios.get).toBeCalledTimes(1);
     // Check the request to get the user
     expect(mockedAxios.get).toBeCalledWith("https://api.coinbase.com/v2/user", {
-      headers: { Authorization: "Bearer cnbstkn294745627362562" },
+      headers: { Authorization: "Bearer cnbstkn294745627362562", Accept: "application/json" },
     });
 
     expect(coinbasePayload).toEqual(