Attachment: treat error if file doesn't exists on storage

Response as 500 Internal Server Error and log it as error.
naver · Aug 14, 2014 · c366aff · c366aff
1 parent df9a8d4
commit c366aff
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/app/controllers/AttachmentApp.java b/app/controllers/AttachmentApp.java
@@ -139,6 +139,11 @@ public static Result getFile(Long id) throws IOException {
 
         File file = attachment.getFile();
 
+        if(file != null && !file.exists()){
+            Logger.error("Attachment ID:" + id + " (" + file.getAbsolutePath() + ") does not exist on storage");
+            return internalServerError("The file does not exist");
+        }
+
         String filename = HttpUtil.encodeContentDisposition(attachment.name);
 
         response().setHeader("Content-Type", attachment.mimeType);