@knime/utils 1.8.0 → 1.9.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @knime/utils
 
+## 1.9.1
+
+### Patch Changes
+
+- dcb9fdd: Fix error rejection for createAbortablePromise util
+
+## 1.9.0
+
+### Minor Changes
+
+- c4bd90d: treat etags in upload as optional, adjust docs
+
 ## 1.8.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knime/utils",
-  "version": "1.8.0",
+  "version": "1.9.1",
   "description": "Internal utility functions and constants shared across @knime packages.",
   "repository": {
     "type": "git",

package/src/promise/createAbortablePromise.ts

@@ -45,7 +45,7 @@ export const createAbortablePromise = () => {
             return;
           }
 
-          reject(error instanceof Error ? error : new Error(error));
+          reject(error);
         });
 
       abortController.signal.addEventListener("abort", abortListener);

package/src/uploadManager/index.ts

@@ -28,6 +28,7 @@ const uploadChunkWithProgress = (params: {
   const { method, url, chunk, chunkIndex, onProgress, abortSignal } = params;
 
   const OK = 200;
+  const CREATED = 201;
 
   return new Promise<{ partId: string }>((resolve, reject) => {
     const xhr = new XMLHttpRequest();
@@ -44,17 +45,13 @@ const uploadChunkWithProgress = (params: {
     };
 
     xhr.onload = () => {
-      if (xhr.status === OK) {
+      if (xhr.status === OK || xhr.status === CREATED) {
         const etag = xhr.getResponseHeader("ETag");
 
-        if (!etag) {
-          reject(new Error("Invalid part upload response"));
-          return;
-        }
-
-        // remove possible doublequotes (") around the etag and/or weak identifier (if either is present)
+        // 1. note that some backends don't return the etag header, then we just set an empty string
+        // 2. remove possible doublequotes (") around the etag and/or weak identifier (if either is present)
         // https://datatracker.ietf.org/doc/html/rfc7232#section-2.3
-        resolve({ partId: etag.replace(/^(?:W\/)?"|"$/g, "") });
+        resolve({ partId: etag?.replace(/^(?:W\/)?"|"$/g, "") ?? "" });
       } else {
         consola.error("Failed or unexpected XHR response", {
           xhrStatus: xhr.status,