use header instead of url param for Battle.net access token

WoWAnalyzer · Oct 29, 2024 · 41c64ca · 41c64ca
1 parent 7d17b91
commit 41c64ca
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/src/route/blizzard/api.ts b/src/route/blizzard/api.ts
@@ -242,13 +242,16 @@ async function fetchApi<T>(
     return undefined;
   }
   const accessToken = await fetchAccessToken(region);
-  const url = makeUrl(region, path, {
-    access_token: accessToken,
-    ...query,
-  });
+  const url = makeUrl(region, path, query);
 
   try {
-    return (await axios.get(url))?.data;
+    return (
+      await axios.get(url, {
+        headers: {
+          Authorization: `Bearer ${accessToken}`,
+        },
+      })
+    )?.data;
   } catch (err: unknown) {
     if (
       err instanceof AxiosError &&