@nxtedition/lib 19.0.40 → 19.0.42

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nxtedition/lib",
-  "version": "19.0.40",
+  "version": "19.0.42",
   "license": "MIT",
   "author": "Robert Nagy <robert.nagy@boffins.se>",
   "type": "module",

package/s3.js

@@ -46,19 +46,19 @@ export async function upload({
 
   assert(queue.concurrency > 0 && queue.concurrency <= 32)
 
+  const ac = new AbortController()
+  const onAbort = () => {
+    ac.abort()
+  }
+  signal?.addEventListener('abort', onAbort)
+
   let uploadId
   try {
-    const multipartUploadOutput = await s3.send(
-      new AWS.CreateMultipartUploadCommand({ Bucket, Key }),
-      { abortSignal: signal },
-    )
-    uploadId = multipartUploadOutput.UploadId
-    logger = logger?.child({ uploadId })
-    logger?.debug('multipart upload created')
-
     const uploader = {
       size: 0,
       hasher: crypto.createHash('md5'),
+      ac,
+      signal: ac.signal,
       part: {
         /** @type {Array<Buffer>} **/ chunks: [],
         hasher: crypto.createHash('md5'),
@@ -67,6 +67,16 @@ export async function upload({
       },
     }
 
+    const multipartUploadOutput = await s3.send(
+      new AWS.CreateMultipartUploadCommand({ Bucket, Key }),
+      { abortSignal: uploader.signal },
+    )
+    uploader.signal.throwIfAborted()
+
+    uploadId = multipartUploadOutput.UploadId
+    logger = logger?.child({ uploadId })
+    logger?.debug('multipart upload created')
+
     const maybeFlush = (minSize) => {
       const { part } = uploader
 
@@ -91,7 +101,7 @@ export async function upload({
       promises.push(
         queue
           .add(
-            async ({ signal }) => {
+            async () => {
               logger?.debug({ number, size }, 'part upload started')
               try {
                 const { ETag } = await s3.send(
@@ -111,20 +121,16 @@ export async function upload({
                       },
                     }),
                   }),
-                  { abortSignal: signal },
                 )
                 logger?.debug({ number, size, etag: ETag }, 'part upload completed')
                 return { part: { ETag, PartNumber: number } }
               } catch (err) {
-                if (err.name === 'AbortError') {
-                  logger?.debug({ err }, 'part upload aborted')
-                } else {
-                  logger?.warn({ err }, 'part upload failed')
-                }
+                uploader.ac.abort(err)
+                logger?.warn({ err }, 'part upload failed')
                 return { error: err }
               }
             },
-            { signal },
+            { signal: uploader.signal },
           )
           .catch((err) => ({ error: err })),
       )
@@ -133,7 +139,7 @@ export async function upload({
     }
 
     for await (const chunk of Body) {
-      signal?.throwIfAborted()
+      uploader.signal.throwIfAborted()
 
       uploader.hasher.update(chunk)
       uploader.size += chunk.byteLength
@@ -145,9 +151,11 @@ export async function upload({
       const thenable = maybeFlush(partSize)
       if (thenable) {
         await thenable
+        uploader.signal.throwIfAborted()
       }
     }
     await maybeFlush()
+    uploader.signal.throwIfAborted()
 
     const parts = []
     const errors = []
@@ -158,7 +166,7 @@ export async function upload({
         parts.push(part)
       }
     }
-    signal?.throwIfAborted()
+    uploader.signal.throwIfAborted()
 
     if (errors.length > 0) {
       throw new AggregateError(errors, 'multipart upload failed')
@@ -175,9 +183,8 @@ export async function upload({
         UploadId: uploadId,
         MultipartUpload: { Parts: parts },
       }),
-      { abortSignal: signal },
     )
-    signal?.throwIfAborted()
+    uploader.signal.throwIfAborted()
 
     const result = {
       size: uploader.size,
@@ -212,5 +219,7 @@ export async function upload({
     }
 
     throw err
+  } finally {
+    signal?.removeEventListener('abort', onAbort)
   }
 }