Even better optimize

Issue: CLDSRV-570
scality · Nov 18, 2024 · c7c7ba1 · c7c7ba1
1 parent e9ae9c2
commit c7c7ba1
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/lib/api/apiUtils/object/abortMultipartUpload.js b/lib/api/apiUtils/object/abortMultipartUpload.js
@@ -104,7 +104,7 @@ function abortMultipartUpload(authInfo, bucketName, objectKey, uploadId, log,
                 bucketName,
                 objectKey,
                 uploadId,
-                versionId: objectMD.versionId
+                versionId: objectMD.versionId,
             });
             return metadata.deleteObjectMD(bucketName, objectKey, { versionId: objectMD.versionId }, log, err => {
                 if (err) {
@@ -125,11 +125,10 @@ function abortMultipartUpload(authInfo, bucketName, objectKey, uploadId, log,
                 return next(null, mpuBucket, storedParts, destBucket);
             }
 
-            const existingLocations = new Set(locations.map(loc => loc.key));
-
             if (objectMD?.location) {
+                const existingLocations = new Set(locations.map(loc => loc.key));
                 const remainingObjectLocations = objectMD.location.filter(loc => !existingLocations.has(loc.key));
-                locations = [...locations, ...remainingObjectLocations];
+                locations.push(...remainingObjectLocations);
             }
 
             return async.eachLimit(locations, 5, (loc, cb) => {