Fix XRPC fetch usage for newly required duplex option (bluesky-social…

…#470) Fix xrpc fetch usage for newly required duplex option
mloar · Jan 18, 2023 · caef814 · caef814
1 parent b718761
commit caef814
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/packages/xrpc/src/client.ts b/packages/xrpc/src/client.ts
@@ -137,11 +137,15 @@ async function defaultFetchHandler(
   httpReqBody: unknown,
 ): Promise<FetchHandlerResponse> {
   try {
-    const res = await fetch(httpUri, {
+    // The duplex field is now required for streaming bodies, but not yet reflected
+    // anywhere in docs or types. See whatwg/fetch#1438, nodejs/node#46221.
+    const reqInit: RequestInit & { duplex: string } = {
       method: httpMethod,
       headers: httpHeaders,
       body: encodeMethodCallBody(httpHeaders, httpReqBody),
-    })
+      duplex: 'half',
+    }
+    const res = await fetch(httpUri, reqInit)
     const resBody = await res.arrayBuffer()
     return {
       status: res.status,