fix(core/js-sdk) Add the credentials field in the fetch() only if sup…

…ported
medusajs · Dec 5, 2024 · be65444 · be65444
1 parent 223bcff
commit be65444
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/packages/core/js-sdk/src/client.ts b/packages/core/js-sdk/src/client.ts
@@ -58,11 +58,17 @@ const normalizeRequest = (
     body = JSON.stringify(body)
   }
 
+  const isFetchCredentialsSupported = "credentials" in Request.prototype
+
   return {
     ...init,
     headers,
     // TODO: Setting this to "include" poses some security risks, as it will send cookies to any domain. We should consider making this configurable.
-    credentials: config.auth?.type === "session" ? "include" : "omit",
+    credentials: isFetchCredentialsSupported
+      ? config.auth?.type === "session"
+        ? "include"
+        : "omit"
+      : undefined,
     ...(body ? { body: body as RequestInit["body"] } : {}),
   } as RequestInit
 }