From 6c6073232985f786cab2ed70749db7ee2212337a Mon Sep 17 00:00:00 2001
From: Daniel da Silva <danielfdsilva@gmail.com>
Date: Thu, 3 Jul 2025 16:59:52 +0100
Subject: [PATCH] Temporarily removes Content-Type header from GET requests

Addresses an issue where the API server rejects GET requests
containing a 'Content-Type: application/json' header.

This is a temporary workaround to ensure compatibility with the
current stac-react implementation and the API server's
requirements.  Once stac-react is updated to handle headers
correctly, this workaround will be removed.
---
 packages/client/src/main.tsx | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/packages/client/src/main.tsx b/packages/client/src/main.tsx
index 04610f0..04b1353 100644
--- a/packages/client/src/main.tsx
+++ b/packages/client/src/main.tsx
@@ -81,3 +81,28 @@ Object.defineProperty(Array.prototype, 'last', {
   },
   set: undefined
 });
+
+// 😱😱😱😱😱😱😱
+//
+// stac-manager relies on stac-react for making STAC API requests.
+// Currently, stac-react adds a 'Content-Type: application/json' header to all
+// requests, including GET requests. However, the api server rejects GET
+// requests with this header. Ideally, the server should accept this header, but
+// as a workaround, we remove it from GET requests to avoid server errors. This
+// is a temporary fix is just to get things working while stac-react is updated.
+// I promise it is temporary!
+const fetch = window.fetch;
+window.fetch = (async (input: RequestInfo, init?: RequestInit) => {
+  // If is a GET request, without body, remove the Content-Type header.
+  if (
+    init &&
+    (!init.method || init.method.toUpperCase() === 'GET') &&
+    !init.body &&
+    // @ts-expect-error Content-Type can be a header property
+    init.headers?.['Content-Type']
+  ) {
+    // @ts-expect-error Content-Type can be a header property
+    delete init.headers['Content-Type'];
+  }
+  return fetch(input, init);
+}) as typeof window.fetch;