Fix compatibility with Unidici

Fixes #467
sindresorhus · Dec 29, 2022 · ae2fe07 · ae2fe07
1 parent aaec57a
commit ae2fe07
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/source/core/Ky.ts b/source/core/Ky.ts
@@ -170,13 +170,13 @@ export class Ky {
 			this._options.signal = this.abortController.signal;
 		}
 
-		this.request = new globalThis.Request(this._input as RequestInfo, this._options as RequestInit);
-
 		if (supportsRequestStreams) {
 			// @ts-expect-error - Types are outdated.
-			this.request.duplex = 'half';
+			this._options.duplex = 'half';
 		}
 
+		this.request = new globalThis.Request(this._input as RequestInfo, this._options as RequestInit);
+
 		if (this._options.searchParams) {
 			// eslint-disable-next-line unicorn/prevent-abbreviations
 			const textSearchParams = typeof this._options.searchParams === 'string'
@@ -194,7 +194,8 @@ export class Ky {
 				this.request.headers.delete('content-type');
 			}
 
-			this.request = new globalThis.Request(new globalThis.Request(url, this.request), this._options as RequestInit);
+			// The spread of `this.request` is required as otherwise it misses the `duplex` option for some reason and throws.
+			this.request = new globalThis.Request(new globalThis.Request(url, {...this.request}), this._options as RequestInit);
 		}
 
 		if (this._options.json !== undefined) {