refactor

graphql · Jan 23, 2025 · edb63e8 · edb63e8
1 parent 863ae8c
commit edb63e8
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/packages/graphiql-toolkit/src/create-fetcher/lib.ts b/packages/graphiql-toolkit/src/create-fetcher/lib.ts
@@ -198,14 +198,14 @@ export async function getWsFetcher(
     return createWebsocketsFetcherFromClient(options.wsClient);
   }
   if (options.subscriptionUrl) {
-    // todo: If there are headers with multiple values, they will be lost. Is this a problem?
-    const fetcherOptsHeadersRecord = Object.fromEntries(
-      // @ts-expect-error: todo enable ES target that has entries on headers
-      new Headers(fetcherOpts?.headers ?? {}).entries(),
-    );
+    // @ts-expect-error: todo enable ES target that has entries on headers
+    const fetcherOptsHeaders = new Headers(
+      fetcherOpts?.headers ?? {},
+    ).entries();
+
     return createWebsocketsFetcherFromUrl(options.subscriptionUrl, {
       ...options.wsConnectionParams,
-      ...fetcherOptsHeadersRecord,
+      ...Object.fromEntries(fetcherOptsHeaders),
     });
   }
   const legacyWebsocketsClient = options.legacyClient || options.legacyWsClient;