core: import: handle error message for invalid zip

packages/fps-importer/handler.ts

@@ -111,7 +111,12 @@ class FpsProblemImportHandler extends Handler {
         } catch (e) {
             if (e instanceof FileTooLargeError) throw e;
             console.log(e);
-            const zip = new AdmZip(this.request.files.file.filepath);
+            let zip: AdmZip;
+            try {
+                zip = new AdmZip(this.request.files.file.filepath);
+            } catch (e) {
+                throw new ValidationError('zip', null, e.message);
+            }
             for (const entry of zip.getEntries()) {
                 try {
                     const buf = entry.getData();

packages/hydrooj/src/handler/import.ts

@@ -37,7 +37,12 @@ class ProblemImportHydroHandler extends Handler {
         if (keepUser) this.checkPriv(PRIV.PRIV_EDIT_SYSTEM);
         if (!this.request.files.file) throw new ValidationError('file');
         const tmpdir = path.join(os.tmpdir(), 'hydro', `${Math.random()}.import`);
-        const zip = new AdmZip(this.request.files.file.filepath);
+        let zip: AdmZip;
+        try {
+            zip = new AdmZip(this.request.files.file.filepath);
+        } catch (e) {
+            throw new ValidationError('zip', null, e.message);
+        }
         await new Promise((resolve, reject) => {
             zip.extractAllToAsync(tmpdir, true, (err) => {
                 if (err) reject(err);

packages/hydrooj/src/handler/problem.ts

@@ -727,7 +727,12 @@ export class ProblemFilesHandler extends ProblemDetailHandler {
         if (!this.user.own(this.pdoc, PERM.PERM_EDIT_PROBLEM_SELF)) this.checkPerm(PERM.PERM_EDIT_PROBLEM);
         const files = [];
         if (filename.endsWith('.zip') && type === 'testdata') {
-            const zip = new AdmZip(this.request.files.file.filepath);
+            let zip: AdmZip;
+            try {
+                zip = new AdmZip(this.request.files.file.filepath);
+            } catch (e) {
+                throw new ValidationError('zip', null, e.message);
+            }
             const entries = zip.getEntries();
             for (const entry of entries) {
                 if (!entry.name) continue;

packages/import-qduoj/handler.ts

@@ -15,7 +15,12 @@ fs.ensureDirSync('/tmp/hydro/import-qduoj');
 
 class ImportQduojHandler extends Handler {
     async fromFile(domainId: string, zipfile: string) {
-        const zip = new AdmZip(zipfile);
+        let zip: AdmZip;
+        try {
+            zip = new AdmZip(zipfile);
+        } catch (e) {
+            throw new ValidationError('zip', null, e.message);
+        }
         const tmp = path.resolve(os.tmpdir(), 'hydro', 'import-qduoj', String.random(32));
         await new Promise((resolve, reject) => {
             zip.extractAllToAsync(tmp, true, (err) => (err ? reject(err) : resolve(null)));