fix: rm tmp dir after file download

obiba · Jan 25, 2025 · d8c8892 · d8c8892
1 parent 81b48ff
commit d8c8892
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/opal-core-ws/src/main/java/org/obiba/opal/web/FilesResource.java b/opal-core-ws/src/main/java/org/obiba/opal/web/FilesResource.java
@@ -14,6 +14,7 @@
 import com.google.common.collect.Lists;
 import jakarta.annotation.Nullable;
 import jakarta.ws.rs.core.*;
+import org.apache.commons.io.FileUtils;
 import org.apache.commons.io.FilenameUtils;
 import org.apache.commons.vfs2.*;
 import org.apache.shiro.SecurityUtils;
@@ -605,7 +606,7 @@ private Response getFile(FileObject file, String key) throws IOException {
       }
       Files.copy(output.toPath(), os);
       if (!Strings.isNullOrEmpty(key)) {
-        output.getParentFile().delete();
+        FileUtils.deleteQuietly(output.getParentFile());
       }
     };
 
@@ -638,7 +639,7 @@ private Response getFolder(FileObject folder, Collection<String> children, Strin
         return true;
       }, new File(tmpDir, fileName), key);
       Files.copy(output.toPath(), os);
-      tmpDir.delete();
+      FileUtils.deleteQuietly(tmpDir);
     };
 
     return Response.ok(stream, mimeType)