Эх сурвалжийг харах

Stop waiting for finalize

Finalize promise will only resolve when the archive is closed. We want
to stream as soon as we have the data so this does not suit us. We want
to log errors that are thrown due to finalize, these should be
propogated by archiver to the response already.

GitOrigin-RevId: 4f9d727a00ead1be3caee62e1e0adba069a545c7
andrew rumble 2 жил өмнө
parent
commit
d4911ea246

+ 6 - 1
services/clsi/app/js/OutputFileArchiveManager.js

@@ -65,7 +65,12 @@ module.exports = {
       })
     }
 
-    await archive.finalize()
+    archive.finalize().catch(error => {
+      logger.error(
+        { error, projectId, userId, build },
+        'error finalizing output files archive'
+      )
+    })
 
     return archive
   },

+ 1 - 1
services/clsi/test/unit/js/OutputFileArchiveManagerTests.js

@@ -31,7 +31,7 @@ describe('OutputFileArchiveManager', function () {
 
     this.archive = {
       append: sinon.stub(),
-      finalize: sinon.stub(),
+      finalize: sinon.stub().resolves(),
       on: sinon.stub(),
     }