@upyo/mailgun 0.5.0-dev.164 → 0.5.0-dev.168

package/dist/index.cjs

@@ -149,18 +149,18 @@ var MailgunHttpClient = class {
 		for (const [key, value] of Object.entries(this.config.headers)) headers.set(key, value);
 		const controller = new AbortController();
 		const timeoutId = setTimeout(() => controller.abort(), this.config.timeout);
-		let signal = controller.signal;
-		if (options.signal) signal = AbortSignal.any([controller.signal, options.signal]);
+		const combinedSignal = combineSignals(controller.signal, options.signal);
 		try {
 			return await globalThis.fetch(url, {
 				...options,
 				headers,
-				signal
+				signal: combinedSignal.signal
 			});
 		} catch (error) {
 			if (isAbortError$1(error) && controller.signal.aborted && !options.signal?.aborted) throw new Error(`Mailgun API request timed out after ${this.config.timeout} ms.`);
 			throw error;
 		} finally {
+			combinedSignal.cleanup();
 			clearTimeout(timeoutId);
 		}
 	}
@@ -203,12 +203,12 @@ function isAbortError$1(error) {
 	return error instanceof Error && error.name === "AbortError";
 }
 function sleep(milliseconds, signal) {
-	if (signal?.aborted) return Promise.reject(createAbortError());
+	if (signal?.aborted) return Promise.reject(createAbortError(signal));
 	return new Promise((resolve, reject) => {
 		function abort() {
 			clearTimeout(timeoutId);
 			signal?.removeEventListener("abort", abort);
-			reject(createAbortError());
+			reject(createAbortError(signal));
 		}
 		const timeoutId = setTimeout(() => {
 			signal?.removeEventListener("abort", abort);
@@ -217,8 +217,41 @@ function sleep(milliseconds, signal) {
 		signal?.addEventListener("abort", abort, { once: true });
 	});
 }
-function createAbortError() {
-	return new DOMException("The operation was aborted.", "AbortError");
+function createAbortError(signal) {
+	return signal?.reason ?? new DOMException("The operation was aborted.", "AbortError");
+}
+function combineSignals(timeoutSignal, externalSignal) {
+	if (externalSignal == null) return {
+		signal: timeoutSignal,
+		cleanup: () => {}
+	};
+	if (typeof AbortSignal.any === "function") return {
+		signal: AbortSignal.any([timeoutSignal, externalSignal]),
+		cleanup: () => {}
+	};
+	const controller = new AbortController();
+	const cleanup = () => {
+		timeoutSignal.removeEventListener("abort", abortFromTimeout);
+		externalSignal.removeEventListener("abort", abortFromExternal);
+	};
+	const abortFromTimeout = () => {
+		cleanup();
+		controller.abort(timeoutSignal.reason);
+	};
+	const abortFromExternal = () => {
+		cleanup();
+		controller.abort(externalSignal.reason);
+	};
+	if (timeoutSignal.aborted) controller.abort(timeoutSignal.reason);
+	else if (externalSignal.aborted) controller.abort(externalSignal.reason);
+	else {
+		timeoutSignal.addEventListener("abort", abortFromTimeout, { once: true });
+		externalSignal.addEventListener("abort", abortFromExternal, { once: true });
+	}
+	return {
+		signal: controller.signal,
+		cleanup
+	};
 }
 function truncateErrorBody(text) {
 	return text.length > 500 ? `${text.slice(0, 500)}...` : text;
@@ -409,7 +442,7 @@ var MailgunTransport = class {
 				provider: "mailgun"
 			};
 		} catch (error) {
-			if (isAbortError(error) && options?.signal?.aborted) throw error;
+			if (isCallerAbort(error, options?.signal)) throw error;
 			const errorMessage = error instanceof Error ? error.message : String(error);
 			return createMailgunFailure(errorMessage, error);
 		}
@@ -488,6 +521,9 @@ function createMailgunFailure(message, error) {
 	});
 	return (0, __upyo_core.createFailedReceipt)(message, { provider: "mailgun" });
 }
+function isCallerAbort(error, signal) {
+	return signal?.aborted === true && (isAbortError(error) || error === signal.reason);
+}
 function isAbortError(error) {
 	return error instanceof Error && error.name === "AbortError";
 }

package/dist/index.js

@@ -126,18 +126,18 @@ var MailgunHttpClient = class {
 		for (const [key, value] of Object.entries(this.config.headers)) headers.set(key, value);
 		const controller = new AbortController();
 		const timeoutId = setTimeout(() => controller.abort(), this.config.timeout);
-		let signal = controller.signal;
-		if (options.signal) signal = AbortSignal.any([controller.signal, options.signal]);
+		const combinedSignal = combineSignals(controller.signal, options.signal);
 		try {
 			return await globalThis.fetch(url, {
 				...options,
 				headers,
-				signal
+				signal: combinedSignal.signal
 			});
 		} catch (error) {
 			if (isAbortError$1(error) && controller.signal.aborted && !options.signal?.aborted) throw new Error(`Mailgun API request timed out after ${this.config.timeout} ms.`);
 			throw error;
 		} finally {
+			combinedSignal.cleanup();
 			clearTimeout(timeoutId);
 		}
 	}
@@ -180,12 +180,12 @@ function isAbortError$1(error) {
 	return error instanceof Error && error.name === "AbortError";
 }
 function sleep(milliseconds, signal) {
-	if (signal?.aborted) return Promise.reject(createAbortError());
+	if (signal?.aborted) return Promise.reject(createAbortError(signal));
 	return new Promise((resolve, reject) => {
 		function abort() {
 			clearTimeout(timeoutId);
 			signal?.removeEventListener("abort", abort);
-			reject(createAbortError());
+			reject(createAbortError(signal));
 		}
 		const timeoutId = setTimeout(() => {
 			signal?.removeEventListener("abort", abort);
@@ -194,8 +194,41 @@ function sleep(milliseconds, signal) {
 		signal?.addEventListener("abort", abort, { once: true });
 	});
 }
-function createAbortError() {
-	return new DOMException("The operation was aborted.", "AbortError");
+function createAbortError(signal) {
+	return signal?.reason ?? new DOMException("The operation was aborted.", "AbortError");
+}
+function combineSignals(timeoutSignal, externalSignal) {
+	if (externalSignal == null) return {
+		signal: timeoutSignal,
+		cleanup: () => {}
+	};
+	if (typeof AbortSignal.any === "function") return {
+		signal: AbortSignal.any([timeoutSignal, externalSignal]),
+		cleanup: () => {}
+	};
+	const controller = new AbortController();
+	const cleanup = () => {
+		timeoutSignal.removeEventListener("abort", abortFromTimeout);
+		externalSignal.removeEventListener("abort", abortFromExternal);
+	};
+	const abortFromTimeout = () => {
+		cleanup();
+		controller.abort(timeoutSignal.reason);
+	};
+	const abortFromExternal = () => {
+		cleanup();
+		controller.abort(externalSignal.reason);
+	};
+	if (timeoutSignal.aborted) controller.abort(timeoutSignal.reason);
+	else if (externalSignal.aborted) controller.abort(externalSignal.reason);
+	else {
+		timeoutSignal.addEventListener("abort", abortFromTimeout, { once: true });
+		externalSignal.addEventListener("abort", abortFromExternal, { once: true });
+	}
+	return {
+		signal: controller.signal,
+		cleanup
+	};
 }
 function truncateErrorBody(text) {
 	return text.length > 500 ? `${text.slice(0, 500)}...` : text;
@@ -386,7 +419,7 @@ var MailgunTransport = class {
 				provider: "mailgun"
 			};
 		} catch (error) {
-			if (isAbortError(error) && options?.signal?.aborted) throw error;
+			if (isCallerAbort(error, options?.signal)) throw error;
 			const errorMessage = error instanceof Error ? error.message : String(error);
 			return createMailgunFailure(errorMessage, error);
 		}
@@ -465,6 +498,9 @@ function createMailgunFailure(message, error) {
 	});
 	return createFailedReceipt(message, { provider: "mailgun" });
 }
+function isCallerAbort(error, signal) {
+	return signal?.aborted === true && (isAbortError(error) || error === signal.reason);
+}
 function isAbortError(error) {
 	return error instanceof Error && error.name === "AbortError";
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@upyo/mailgun",
-  "version": "0.5.0-dev.164",
+  "version": "0.5.0-dev.168",
   "description": "Mailgun transport for Upyo email library",
   "keywords": [
     "email",
@@ -53,7 +53,7 @@
   },
   "sideEffects": false,
   "peerDependencies": {
-    "@upyo/core": "0.5.0-dev.164+5e283c64"
+    "@upyo/core": "0.5.0-dev.168+1e808a3a"
   },
   "devDependencies": {
     "tsdown": "^0.12.7",