@eik/core 2.1.58 → 2.1.59

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [2.1.59](https://github.com/eik-lib/core/compare/v2.1.58...v2.1.59) (2026-06-09)
+
+
+### Bug Fixes
+
+* Improve unpacking files ([#598](https://github.com/eik-lib/core/issues/598)) ([41cd672](https://github.com/eik-lib/core/commit/41cd672db44af05e06acd2dc047bffb0ed2006a7))
+
 ## [2.1.58](https://github.com/eik-lib/core/compare/v2.1.57...v2.1.58) (2026-06-09)
 
 

package/lib/multipart/parser.js

@@ -93,9 +93,12 @@ const MultipartParser = class MultipartParser {
 									done(item);
 								})
 								.catch((error) => {
-									// Emit an error on busboy instead of rejecting
-									// This will break and terminate the stream stright away
+									// Emit an error on busboy instead of rejecting.
+									// This breaks and terminates the stream straight away.
 									busboy.emit("error", error);
+									// Settle this wrapper Promise so Promise.all(queue)
+									// in the close handler can always complete.
+									done(null);
 								});
 						}),
 					);
@@ -142,6 +145,7 @@ const MultipartParser = class MultipartParser {
 				`multipart - Start extracting package - Field: ${fieldname} - Filename: ${filename}`,
 			);
 
+			const ac = new AbortController();
 			/** @type {Promise<any>[]} */
 			const queue = [];
 
@@ -149,6 +153,7 @@ const MultipartParser = class MultipartParser {
 				this._log.info(
 					`multipart - Uploaded package exceeds legal file size limit - Field: ${fieldname} - Filename: ${filename}`,
 				);
+				ac.abort();
 				file.emit("error", new HttpError.PayloadTooLarge());
 			});
 
@@ -161,7 +166,15 @@ const MultipartParser = class MultipartParser {
 						entry.resume();
 						return;
 					}
-					queue.push(this._persistFile({ incoming, entry }));
+					const persist = this._persistFile({
+						incoming,
+						entry,
+						signal: ac.signal,
+					});
+					// Suppress unhandled rejections from writes aborted due to
+					// an upstream error — they are intentionally abandoned.
+					persist.catch(() => {});
+					queue.push(persist);
 				},
 			});
 
@@ -172,6 +185,8 @@ const MultipartParser = class MultipartParser {
 					);
 					this._log.trace(error);
 
+					ac.abort();
+
 					switch (error.code) {
 						case "TAR_BAD_ARCHIVE":
 						case "TAR_ABORT":
@@ -201,61 +216,71 @@ const MultipartParser = class MultipartParser {
 	}
 
 	/**
-	 * @param {{ incoming: any, entry: any }} options
+	 * @param {{ incoming: any, entry: any, signal: AbortSignal }} options
 	 */
-	_persistFile({ incoming, entry }) {
-		// eslint-disable-next-line no-async-promise-executor
-		return new Promise(async (resolve, reject) => {
-			const asset = new Asset({
-				pathname: entry.path,
-				version: incoming.version,
-				name: incoming.name,
-				type: incoming.type,
-				org: incoming.org,
-			});
-			asset.size = entry.size;
+	async _persistFile({ incoming, entry, signal }) {
+		const asset = new Asset({
+			pathname: entry.path,
+			version: incoming.version,
+			name: incoming.name,
+			type: incoming.type,
+			org: incoming.org,
+		});
+		asset.size = entry.size;
 
-			const path = createFilePathToAsset(asset);
+		const path = createFilePathToAsset(asset);
 
-			this._log.info(
-				`multipart - Start writing asset to sink - Pathname: ${path}`,
-			);
+		this._log.info(
+			`multipart - Start writing asset to sink - Pathname: ${path}`,
+		);
 
-			try {
-				if (!this._sink) {
-					reject(new Error("No sink configured"));
-					return;
-				}
-				const writer = await this._sink.write(path, asset.mimeType);
+		if (!this._sink) {
+			throw new Error("No sink configured");
+		}
+		const writer = await this._sink.write(path, asset.mimeType);
 
-				const integrityStream = ssri.integrityStream({ single: true });
-				let hash = "";
-				integrityStream.once("integrity", (/** @type {any} */ integrity) => {
-					hash = integrity;
-				});
+		const integrityStream = ssri.integrityStream({ single: true });
+		let hash = "";
+		integrityStream.once("integrity", (/** @type {any} */ integrity) => {
+			hash = integrity;
+		});
 
-				pipeline(entry, integrityStream, writer, (error) => {
-					if (error) {
-						this._log.error(
-							`multipart - Failed writing asset to sink - Pathname: ${path}`,
-						);
-						this._log.trace(error);
+		return new Promise((resolve, reject) => {
+			const onAbort = () => {
+				entry.destroy();
+				integrityStream.destroy();
+				writer.destroy();
+			};
 
-						reject(error);
-						return;
-					}
+			if (signal.aborted) {
+				onAbort();
+				reject(new Error("Aborted"));
+				return;
+			}
 
-					asset.integrity = hash.toString();
+			signal.addEventListener("abort", onAbort, { once: true });
 
-					this._log.info(
-						`multipart - Successfully wrote asset to sink - Pathname: ${path}`,
+			pipeline(entry, integrityStream, writer, (error) => {
+				signal.removeEventListener("abort", onAbort);
+
+				if (error) {
+					this._log.error(
+						`multipart - Failed writing asset to sink - Pathname: ${path}`,
 					);
+					this._log.trace(error);
 
-					resolve(asset);
-				});
-			} catch (error) {
-				reject(error);
-			}
+					reject(error);
+					return;
+				}
+
+				asset.integrity = hash.toString();
+
+				this._log.info(
+					`multipart - Successfully wrote asset to sink - Pathname: ${path}`,
+				);
+
+				resolve(asset);
+			});
 		});
 	}
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eik/core",
-  "version": "2.1.58",
+  "version": "2.1.59",
   "description": "Core server package",
   "main": "lib/main.js",
   "types": "./types/main.d.ts",

package/types/handlers/alias.post.d.ts

@@ -51,9 +51,10 @@ declare const AliasPost: {
                 filename: any;
                 incoming: any;
             }): Promise<any>;
-            _persistFile({ incoming, entry }: {
+            _persistFile({ incoming, entry, signal }: {
                 incoming: any;
                 entry: any;
+                signal: AbortSignal;
             }): Promise<any>;
             get [Symbol.toStringTag](): string;
         };

package/types/handlers/alias.put.d.ts

@@ -51,9 +51,10 @@ declare const AliasPut: {
                 filename: any;
                 incoming: any;
             }): Promise<any>;
-            _persistFile({ incoming, entry }: {
+            _persistFile({ incoming, entry, signal }: {
                 incoming: any;
                 entry: any;
+                signal: AbortSignal;
             }): Promise<any>;
             get [Symbol.toStringTag](): string;
         };

package/types/handlers/auth.post.d.ts

@@ -51,9 +51,10 @@ declare const AuthPost: {
                 filename: any;
                 incoming: any;
             }): Promise<any>;
-            _persistFile({ incoming, entry }: {
+            _persistFile({ incoming, entry, signal }: {
                 incoming: any;
                 entry: any;
+                signal: AbortSignal;
             }): Promise<any>;
             get [Symbol.toStringTag](): string;
         };

package/types/handlers/pkg.put.d.ts

@@ -54,9 +54,10 @@ declare const PkgPut: {
                 filename: any;
                 incoming: any;
             }): Promise<any>;
-            _persistFile({ incoming, entry }: {
+            _persistFile({ incoming, entry, signal }: {
                 incoming: any;
                 entry: any;
+                signal: AbortSignal;
             }): Promise<any>;
             get [Symbol.toStringTag](): string;
         };

package/types/multipart/parser.d.ts

@@ -52,11 +52,12 @@ declare const MultipartParser: {
             incoming: any;
         }): Promise<any>;
         /**
-         * @param {{ incoming: any, entry: any }} options
+         * @param {{ incoming: any, entry: any, signal: AbortSignal }} options
          */
-        _persistFile({ incoming, entry }: {
+        _persistFile({ incoming, entry, signal }: {
             incoming: any;
             entry: any;
+            signal: AbortSignal;
         }): Promise<any>;
         get [Symbol.toStringTag](): string;
     };