Merge pull request #204 from hearchco/as/fix/fetch-headers

fix(fetch): add headers to server fetch
hearchco · Apr 3, 2024 · 908c5f1 · 908c5f1
2 parents 270b725 + d2004e4
commit 908c5f1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/lib/functions/fetchPublicResultsJSON.ts b/src/lib/functions/fetchPublicResultsJSON.ts
@@ -14,7 +14,10 @@ export async function fetchPublicResultsJSON(params: URLSearchParams): Promise<R
 
 	let response: Response;
 	try {
-		response = await fetch(apiUrl);
+		response = await fetch(apiUrl, {
+			method: 'GET' // POST doesn't cache on CDN
+			// no need for headers since it runs in the browser
+		});
 	} catch (err: any) {
 		// Service Unavailable
 		throw error(503, `Failed to fetch results: ${err.message}`);

diff --git a/src/lib/functions/fetchResultsJSON.ts b/src/lib/functions/fetchResultsJSON.ts
@@ -23,7 +23,14 @@ export async function fetchResultsJSON(
 
 	let response: Response;
 	try {
-		response = await fetch(apiUrl);
+		response = await fetch(apiUrl, {
+			// must set headers since it runs on the server
+			method: 'GET', // POST doesn't cache on CDN
+			headers: {
+				Accept: 'application/json',
+				'Accept-Encoding': 'gzip, deflate, br'
+			}
+		});
 	} catch (err: any) {
 		await delayed;
 		// Service Unavailable