From 28b746efcad89de2237a29faeea62c33b6ba4ef7 Mon Sep 17 00:00:00 2001
From: Lucian <lucian@gitcoin.co>
Date: Tue, 10 Dec 2024 01:09:33 -0700
Subject: [PATCH] fix(platforms): explicitly request json response from
 coinbase API (#3121)

---
 platforms/src/Coinbase/Providers/coinbase.ts      | 3 +--
 platforms/src/Coinbase/__tests__/coinbase.test.ts | 8 ++++----
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/platforms/src/Coinbase/Providers/coinbase.ts b/platforms/src/Coinbase/Providers/coinbase.ts
index f09b37872c..d103221281 100644
--- 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
index d22bb0afb3..3e4e0389f2 100644
--- 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(