happy-dom 17.6.2 → 18.0.0

package/src/fetch/Fetch.ts

@@ -86,7 +86,7 @@ export default class Fetch {
 		url: IRequestInfo;
 		init?: IRequestInit;
 		redirectCount?: number;
-		contentType?: string;
+		contentType?: string | null;
 		disableCache?: boolean;
 		disableSameOriginPolicy?: boolean;
 		unfilteredHeaders?: Headers;
@@ -138,7 +138,7 @@ export default class Fetch {
 			if (this.request.signal[PropertySymbol.reason] !== undefined) {
 				throw this.request.signal[PropertySymbol.reason];
 			}
-			throw new this[PropertySymbol.window].DOMException(
+			throw new this.#window.DOMException(
 				'signal is aborted without reason',
 				DOMExceptionNameEnum.abortError
 			);
@@ -218,7 +218,7 @@ export default class Fetch {
 			const compliesWithCrossOriginPolicy = await this.compliesWithCrossOriginPolicy();
 
 			if (!compliesWithCrossOriginPolicy) {
-				this.#browserFrame?.page?.console.warn(
+				this.#browserFrame.page.console.warn(
 					`Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at "${this.request.url}".`
 				);
 				throw new this.#window.DOMException(
@@ -332,7 +332,7 @@ export default class Fetch {
 		const taskID = this.#browserFrame[PropertySymbol.asyncTaskManager].startTask();
 
 		if (this.request.method !== 'GET') {
-			this.#browserFrame?.page?.console.error(
+			this.#browserFrame.page.console.error(
 				`${this.request.method} ${this.request.url} 404 (Not Found)`
 			);
 			const response = VirtualServerUtility.getNotFoundResponse(this.#window);
@@ -354,7 +354,7 @@ export default class Fetch {
 				stat.isDirectory() ? Path.join(filePath, 'index.html') : filePath
 			);
 		} catch (error) {
-			this.#browserFrame?.page?.console.error(
+			this.#browserFrame.page.console.error(
 				`${this.request.method} ${this.request.url} 404 (Not Found)`
 			);
 
@@ -400,7 +400,7 @@ export default class Fetch {
 			waitingForBody: false
 		};
 
-		response[PropertySymbol.cachedResponse] = this.#browserFrame.page?.context?.responseCache.add(
+		response[PropertySymbol.cachedResponse] = this.#browserFrame.page.context.responseCache.add(
 			this.request,
 			cachedResponse
 		);
@@ -490,7 +490,7 @@ export default class Fetch {
 		const allowMethods: string[] = [];
 
 		if (response.headers.has('Access-Control-Allow-Methods')) {
-			const allowMethodsHeader = response.headers.get('Access-Control-Allow-Methods');
+			const allowMethodsHeader = response.headers.get('Access-Control-Allow-Methods')!;
 			if (allowMethodsHeader !== '*') {
 				for (const method of allowMethodsHeader.split(',')) {
 					allowMethods.push(method.trim().toUpperCase());
@@ -534,7 +534,7 @@ export default class Fetch {
 					response[PropertySymbol.cachedResponse] =
 						this.#browserFrame.page.context.responseCache.add(this.request, {
 							...response,
-							headers: this.responseHeaders,
+							headers: this.responseHeaders!,
 							body: response[PropertySymbol.buffer],
 							waitingForBody: !response[PropertySymbol.buffer] && !!response.body
 						});
@@ -553,7 +553,7 @@ export default class Fetch {
 
 				// The browser outputs errors to the console when the response is not ok.
 				if (returnResponse instanceof Response && !returnResponse.ok) {
-					this.#browserFrame?.page?.console.error(
+					this.#browserFrame.page.console.error(
 						`${this.request.method} ${this.request.url} ${returnResponse.status} (${returnResponse.statusText})`
 					);
 				}
@@ -618,9 +618,11 @@ export default class Fetch {
 					DOMExceptionNameEnum.networkError
 				);
 
-				if (this.response && this.response.body) {
-					this.response.body[PropertySymbol.error] = error;
-					if (!this.response.body.locked) {
+				this.request[PropertySymbol.error] = error;
+
+				if (this.response) {
+					this.response[PropertySymbol.error] = error;
+					if (this.response.body && !this.response.body.locked) {
 						this.response.body.cancel(error);
 					}
 				}
@@ -643,7 +645,7 @@ export default class Fetch {
 		socket.prependListener('close', onSocketClose);
 		socket.on('data', onData);
 
-		this.nodeRequest.on('close', () => {
+		this.nodeRequest!.on('close', () => {
 			socket.removeListener('close', onSocketClose);
 			socket.removeListener('data', onData);
 		});
@@ -666,8 +668,8 @@ export default class Fetch {
 	 */
 	private onError(error: Error): void {
 		this.finalizeRequest();
-		this.#browserFrame?.page?.console.error(error);
-		this.reject(
+		this.#browserFrame.page.console.error(error);
+		this.reject!(
 			new this.#window.DOMException(
 				`Failed to execute "fetch()" on "Window" with URL "${this.request.url}": ${error.message}`,
 				DOMExceptionNameEnum.networkError
@@ -685,9 +687,11 @@ export default class Fetch {
 		);
 
 		this.request[PropertySymbol.aborted] = true;
+		this.request[PropertySymbol.error] = error;
 
-		if (this.request.body) {
-			this.request.body[PropertySymbol.error] = error;
+		if (this.response) {
+			this.response[PropertySymbol.aborted] = true;
+			this.response[PropertySymbol.error] = error;
 		}
 
 		if (this.listeners.onSignalAbort) {
@@ -703,7 +707,6 @@ export default class Fetch {
 		}
 
 		if (this.response && this.response.body) {
-			this.response.body[PropertySymbol.error] = error;
 			if (!this.response.body.locked) {
 				this.response.body.cancel(error);
 			}
@@ -721,7 +724,7 @@ export default class Fetch {
 			nodeResponse.headers['transfer-encoding'] === 'chunked' &&
 			!nodeResponse.headers['content-length'];
 
-		this.nodeRequest.setTimeout(0);
+		this.nodeRequest!.setTimeout(0);
 		this.responseHeaders = FetchResponseHeaderUtility.parseResponseHeaders({
 			browserFrame: this.#browserFrame,
 			requestURL: this.request[PropertySymbol.url],
@@ -736,7 +739,7 @@ export default class Fetch {
 			this.request.signal.removeEventListener('abort', this.listeners.onSignalAbort)
 		);
 
-		let body = Stream.pipeline(nodeResponse, new Stream.PassThrough(), (error: Error) => {
+		let body = Stream.pipeline(nodeResponse, new Stream.PassThrough(), (error) => {
 			if (error) {
 				// Ignore error as it is forwarded to the response body.
 			}
@@ -762,7 +765,7 @@ export default class Fetch {
 			);
 			(<boolean>this.response.redirected) = this.redirectCount > 0;
 			(<string>this.response.url) = this.request.url;
-			this.resolve(this.response);
+			this.resolve!(this.response);
 			return;
 		}
 
@@ -776,7 +779,7 @@ export default class Fetch {
 				finishFlush: Zlib.constants.Z_SYNC_FLUSH
 			};
 
-			body = Stream.pipeline(body, Zlib.createGunzip(zlibOptions), (error: Error) => {
+			body = Stream.pipeline(body, Zlib.createGunzip(zlibOptions), (error) => {
 				if (error) {
 					// Ignore error as it is forwarded to the response body.
 				}
@@ -787,7 +790,7 @@ export default class Fetch {
 			);
 			(<boolean>this.response.redirected) = this.redirectCount > 0;
 			(<string>this.response.url) = this.request.url;
-			this.resolve(this.response);
+			this.resolve!(this.response);
 			return;
 		}
 
@@ -822,7 +825,7 @@ export default class Fetch {
 				);
 				(<boolean>this.response.redirected) = this.redirectCount > 0;
 				(<string>this.response.url) = this.request.url;
-				this.resolve(this.response);
+				this.resolve!(this.response);
 			});
 			raw.on('end', () => {
 				// Some old IIS servers return zero-length OK deflate responses, so 'data' is never emitted.
@@ -833,7 +836,7 @@ export default class Fetch {
 					);
 					(<boolean>this.response.redirected) = this.redirectCount > 0;
 					(<string>this.response.url) = this.request.url;
-					this.resolve(this.response);
+					this.resolve!(this.response);
 				}
 			});
 			return;
@@ -852,7 +855,7 @@ export default class Fetch {
 			);
 			(<boolean>this.response.redirected) = this.redirectCount > 0;
 			(<string>this.response.url) = this.request.url;
-			this.resolve(this.response);
+			this.resolve!(this.response);
 			return;
 		}
 
@@ -863,7 +866,7 @@ export default class Fetch {
 		);
 		(<boolean>this.response.redirected) = this.redirectCount > 0;
 		(<string>this.response.url) = this.request.url;
-		this.resolve(this.response);
+		this.resolve!(this.response);
 	}
 
 	/**
@@ -874,14 +877,14 @@ export default class Fetch {
 	 * @returns True if redirect response was handled, false otherwise.
 	 */
 	private handleRedirectResponse(nodeResponse: IncomingMessage, responseHeaders: Headers): boolean {
-		if (!FetchResponseRedirectUtility.isRedirect(nodeResponse.statusCode)) {
+		if (!FetchResponseRedirectUtility.isRedirect(nodeResponse.statusCode!)) {
 			return false;
 		}
 
 		switch (this.request.redirect) {
 			case 'error':
 				this.finalizeRequest();
-				this.reject(
+				this.reject!(
 					new this.#window.DOMException(
 						`URI requested responds with a redirect, redirect mode is set to "error": ${this.request.url}`,
 						DOMExceptionNameEnum.abortError
@@ -897,14 +900,14 @@ export default class Fetch {
 					nodeResponse.statusCode === 303 ||
 					((nodeResponse.statusCode === 301 || nodeResponse.statusCode === 302) &&
 						this.request.method === 'POST');
-				let locationURL: URL = null;
+				let locationURL: URL | null = null;
 
 				if (locationHeader !== null) {
 					try {
 						locationURL = new URL(locationHeader, this.request.url);
 					} catch {
 						this.finalizeRequest();
-						this.reject(
+						this.reject!(
 							new this.#window.DOMException(
 								`URI requested responds with an invalid redirect URL: ${locationHeader}`,
 								DOMExceptionNameEnum.uriMismatchError
@@ -920,7 +923,7 @@ export default class Fetch {
 
 				if (FetchResponseRedirectUtility.isMaxRedirectsReached(this.redirectCount)) {
 					this.finalizeRequest();
-					this.reject(
+					this.reject!(
 						new this.#window.DOMException(
 							`Maximum redirects reached at: ${this.request.url}`,
 							DOMExceptionNameEnum.networkError
@@ -975,20 +978,18 @@ export default class Fetch {
 					url: locationURL,
 					init: requestInit,
 					redirectCount: this.redirectCount + 1,
-					contentType: !shouldBecomeGetRequest
-						? this.request[PropertySymbol.contentType]
-						: undefined
+					contentType: !shouldBecomeGetRequest ? this.request[PropertySymbol.contentType] : null
 				});
 
 				this.finalizeRequest();
 				fetch
 					.send()
-					.then((response) => this.resolve(response))
-					.catch((error) => this.reject(error));
+					.then((response) => this.resolve!(response))
+					.catch((error) => this.reject!(error));
 				return true;
 			default:
 				this.finalizeRequest();
-				this.reject(
+				this.reject!(
 					new this.#window.DOMException(
 						`Redirect option '${this.request.redirect}' is not a valid value of IRequestRedirect`
 					)
@@ -1002,7 +1003,7 @@ export default class Fetch {
 	 */
 	private finalizeRequest(): void {
 		this.request.signal.removeEventListener('abort', this.listeners.onSignalAbort);
-		this.nodeRequest.destroy();
+		this.nodeRequest?.destroy();
 	}
 
 	/**
@@ -1017,9 +1018,11 @@ export default class Fetch {
 		);
 
 		this.request[PropertySymbol.aborted] = true;
+		this.request[PropertySymbol.error] = error;
 
-		if (this.request.body) {
-			this.request.body[PropertySymbol.error] = error;
+		if (this.response) {
+			this.response[PropertySymbol.aborted] = true;
+			this.response[PropertySymbol.error] = error;
 		}
 
 		if (this.nodeRequest && !this.nodeRequest.destroyed) {
@@ -1031,7 +1034,6 @@ export default class Fetch {
 		}
 
 		if (this.response && this.response.body) {
-			this.response.body[PropertySymbol.error] = error;
 			if (!this.response.body.locked) {
 				this.response.body.cancel(error);
 			}

package/src/fetch/Headers.ts

@@ -16,7 +16,7 @@ export default class Headers {
 	 *
 	 * @param init Headers init.
 	 */
-	constructor(init?: IHeadersInit) {
+	constructor(init?: IHeadersInit | null) {
 		if (init) {
 			if (init instanceof Headers) {
 				this[PropertySymbol.entries] = JSON.parse(JSON.stringify(init[PropertySymbol.entries]));

package/src/fetch/Request.ts

@@ -45,6 +45,7 @@ export default class Request implements Request {
 
 	// Internal properties
 	public [PropertySymbol.aborted]: boolean = false;
+	public [PropertySymbol.error]: Error | null = null;
 	public [PropertySymbol.contentLength]: number | null = null;
 	public [PropertySymbol.contentType]: string | null = null;
 	public [PropertySymbol.referrer]: '' | 'no-referrer' | 'client' | URL = 'client';
@@ -100,7 +101,7 @@ export default class Request implements Request {
 		const { stream, buffer, contentType, contentLength } = FetchBodyUtility.getBodyStream(
 			input instanceof Request && (input[PropertySymbol.bodyBuffer] || input.body)
 				? input[PropertySymbol.bodyBuffer] || FetchBodyUtility.cloneBodyStream(window, input)
-				: init?.body
+				: (init?.body ?? null)
 		);
 
 		this[PropertySymbol.bodyBuffer] = buffer;
@@ -300,20 +301,18 @@ export default class Request implements Request {
 			);
 		}
 
-		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager();
+		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager()!;
 
 		this[PropertySymbol.bodyUsed] = true;
 
 		const taskID = asyncTaskManager.startTask(() => {
-			if (this.body) {
-				this.body[PropertySymbol.aborted] = true;
-			}
+			this[PropertySymbol.aborted] = true;
 			this.signal[PropertySymbol.abort]();
 		});
 		let buffer: Buffer;
 
 		try {
-			buffer = await FetchBodyUtility.consumeBodyStream(window, this.body);
+			buffer = await FetchBodyUtility.consumeBodyStream(window, this);
 		} catch (error) {
 			asyncTaskManager.endTask(taskID);
 			throw error;
@@ -351,20 +350,18 @@ export default class Request implements Request {
 			);
 		}
 
-		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager();
+		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager()!;
 
 		this[PropertySymbol.bodyUsed] = true;
 
 		const taskID = asyncTaskManager.startTask(() => {
-			if (this.body) {
-				this.body[PropertySymbol.aborted] = true;
-			}
+			this[PropertySymbol.aborted] = true;
 			this.signal[PropertySymbol.abort]();
 		});
 		let buffer: Buffer;
 
 		try {
-			buffer = await FetchBodyUtility.consumeBodyStream(window, this.body);
+			buffer = await FetchBodyUtility.consumeBodyStream(window, this);
 		} catch (error) {
 			asyncTaskManager.endTask(taskID);
 			throw error;
@@ -390,20 +387,18 @@ export default class Request implements Request {
 			);
 		}
 
-		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager();
+		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager()!;
 
 		this[PropertySymbol.bodyUsed] = true;
 
 		const taskID = asyncTaskManager.startTask(() => {
-			if (this.body) {
-				this.body[PropertySymbol.aborted] = true;
-			}
+			this[PropertySymbol.aborted] = true;
 			this.signal[PropertySymbol.abort]();
 		});
 		let buffer: Buffer;
 
 		try {
-			buffer = await FetchBodyUtility.consumeBodyStream(window, this.body);
+			buffer = await FetchBodyUtility.consumeBodyStream(window, this);
 		} catch (error) {
 			asyncTaskManager.endTask(taskID);
 			throw error;
@@ -431,10 +426,11 @@ export default class Request implements Request {
 	 */
 	public async formData(): Promise<FormData> {
 		const window = this[PropertySymbol.window];
-		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager();
+		const asyncTaskManager = new WindowBrowserContext(window).getAsyncTaskManager()!;
+
 		const contentType = this[PropertySymbol.contentType];
 
-		if (/multipart/i.test(contentType)) {
+		if (this.body && contentType && /multipart/i.test(contentType)) {
 			if (this[PropertySymbol.bodyUsed]) {
 				throw new window.DOMException(
 					`Body has already been used for "${this.url}".`,
@@ -445,19 +441,13 @@ export default class Request implements Request {
 			this[PropertySymbol.bodyUsed] = true;
 
 			const taskID = asyncTaskManager.startTask(() => {
-				if (this.body) {
-					this.body[PropertySymbol.aborted] = true;
-				}
+				this[PropertySymbol.aborted] = true;
 				this.signal[PropertySymbol.abort]();
 			});
 			let formData: FormData;
 
 			try {
-				const result = await MultipartFormDataParser.streamToFormData(
-					window,
-					this.body,
-					contentType
-				);
+				const result = await MultipartFormDataParser.streamToFormData(window, this, contentType);
 				formData = result.formData;
 			} catch (error) {
 				asyncTaskManager.endTask(taskID);

package/src/fetch/ResourceFetch.ts

@@ -40,13 +40,17 @@ export default class ResourceFetch {
 	): Promise<string> {
 		const browserFrame = new WindowBrowserContext(this.window).getBrowserFrame();
 
+		if (!browserFrame) {
+			return '';
+		}
+
 		// Preloaded resource
 		if (destination === 'script' || destination === 'style') {
 			const preloadKey = PreloadUtility.getKey({
 				url: String(url),
 				destination,
 				mode: 'cors',
-				credentialsMode: options.credentials || 'same-origin'
+				credentialsMode: options?.credentials || 'same-origin'
 			});
 			const preloadEntry = this.window.document[PropertySymbol.preloads].get(preloadKey);
 
@@ -55,15 +59,15 @@ export default class ResourceFetch {
 
 				const response = preloadEntry.response || (await preloadEntry.onResponseAvailable());
 
-				if (!response.ok) {
+				if (response && !response.ok) {
 					throw new this.window.DOMException(
 						`Failed to perform request to "${
 							new URL(url, this.window.location.href).href
-						}". Status ${preloadEntry.response.status} ${preloadEntry.response.statusText}.`
+						}". Status ${preloadEntry.response?.status || '0'} ${preloadEntry.response?.statusText || 'Unknown'}.`
 					);
 				}
 
-				return preloadEntry.response[PropertySymbol.buffer].toString();
+				return preloadEntry.response?.[PropertySymbol.buffer]?.toString() || '';
 			}
 		}
 
@@ -109,13 +113,17 @@ export default class ResourceFetch {
 	): string {
 		const browserFrame = new WindowBrowserContext(this.window).getBrowserFrame();
 
+		if (!browserFrame) {
+			return '';
+		}
+
 		// Preloaded resource
 		if (destination === 'script' || destination === 'style') {
 			const preloadKey = PreloadUtility.getKey({
 				url: String(url),
 				destination,
 				mode: 'cors',
-				credentialsMode: options.credentials || 'same-origin'
+				credentialsMode: options?.credentials || 'same-origin'
 			});
 			const preloadEntry = this.window.document[PropertySymbol.preloads].get(preloadKey);
 
@@ -133,6 +141,12 @@ export default class ResourceFetch {
 					);
 				}
 
+				if (!preloadEntry.response[PropertySymbol.buffer]) {
+					throw new this.window.DOMException(
+						`Failed to perform request to "${new URL(url, this.window.location.href).href}". Response buffer is not available.`
+					);
+				}
+
 				return preloadEntry.response[PropertySymbol.buffer].toString();
 			}
 		}
@@ -158,6 +172,6 @@ export default class ResourceFetch {
 			);
 		}
 
-		return response.body.toString();
+		return response.body?.toString() || '';
 	}
 }

package/src/fetch/Response.ts

@@ -43,6 +43,8 @@ export default class Response implements Response {
 	public readonly headers: Headers;
 	public [PropertySymbol.cachedResponse]: ICachedResponse | null = null;
 	public [PropertySymbol.buffer]: Buffer | null = null;
+	public [PropertySymbol.aborted]: boolean = false;
+	public [PropertySymbol.error]: Error | null = null;
 
 	/**
 	 * Constructor.
@@ -119,13 +121,11 @@ export default class Response implements Response {
 
 		if (!buffer) {
 			const taskID = asyncTaskManager.startTask(() => {
-				if (this.body) {
-					this.body[PropertySymbol.aborted] = true;
-				}
+				this[PropertySymbol.aborted] = true;
 			});
 
 			try {
-				buffer = await FetchBodyUtility.consumeBodyStream(window, this.body);
+				buffer = await FetchBodyUtility.consumeBodyStream(window, this);
 			} catch (error) {
 				asyncTaskManager.endTask(taskID);
 				throw error;
@@ -181,12 +181,10 @@ export default class Response implements Response {
 
 		if (!buffer) {
 			const taskID = asyncTaskManager.startTask(() => {
-				if (this.body) {
-					this.body[PropertySymbol.aborted] = true;
-				}
+				this[PropertySymbol.aborted] = true;
 			});
 			try {
-				buffer = await FetchBodyUtility.consumeBodyStream(window, this.body);
+				buffer = await FetchBodyUtility.consumeBodyStream(window, this);
 			} catch (error) {
 				asyncTaskManager.endTask(taskID);
 				throw error;
@@ -229,12 +227,10 @@ export default class Response implements Response {
 
 		if (!buffer) {
 			const taskID = asyncTaskManager.startTask(() => {
-				if (this.body) {
-					this.body[PropertySymbol.aborted] = true;
-				}
+				this[PropertySymbol.aborted] = true;
 			});
 			try {
-				buffer = await FetchBodyUtility.consumeBodyStream(window, this.body);
+				buffer = await FetchBodyUtility.consumeBodyStream(window, this);
 			} catch (error) {
 				asyncTaskManager.endTask(taskID);
 				throw error;
@@ -274,7 +270,7 @@ export default class Response implements Response {
 		const asyncTaskManager = browserFrame[PropertySymbol.asyncTaskManager];
 		const contentType = this.headers.get('Content-Type');
 
-		if (/multipart/i.test(contentType)) {
+		if (contentType && this.body && /multipart/i.test(contentType)) {
 			if (this.bodyUsed) {
 				throw new window.DOMException(
 					`Body has already been used for "${this.url}".`,
@@ -285,19 +281,13 @@ export default class Response implements Response {
 			(<boolean>this.bodyUsed) = true;
 
 			const taskID = browserFrame[PropertySymbol.asyncTaskManager].startTask(() => {
-				if (this.body) {
-					this.body[PropertySymbol.aborted] = true;
-				}
+				this[PropertySymbol.aborted] = true;
 			});
 			let formData: FormData;
 			let buffer: Buffer;
 
 			try {
-				const result = await MultipartFormDataParser.streamToFormData(
-					window,
-					this.body,
-					contentType
-				);
+				const result = await MultipartFormDataParser.streamToFormData(window, this, contentType);
 				formData = result.formData;
 				buffer = result.buffer;
 			} catch (error) {