From 0a95b5594b654dbec73a9e50e340f4975aa3e8e5 Mon Sep 17 00:00:00 2001
From: Francis Dinh <normandy@firemail.cc>
Date: Thu, 14 Jun 2018 12:12:38 -0400
Subject: [PATCH] Add missing file extension if file does not have one

---
 lib/pleroma/upload.ex | 29 ++++++++++++++++++++++++-----
 test/upload_test.exs  | 13 +++++++++++++
 2 files changed, 37 insertions(+), 5 deletions(-)

diff --git a/lib/pleroma/upload.ex b/lib/pleroma/upload.ex
index ab4bd16f0..e62ab820a 100644
--- a/lib/pleroma/upload.ex
+++ b/lib/pleroma/upload.ex
@@ -86,10 +86,15 @@ def upload_path do
   end
 
   defp create_name(uuid, ext, type) do
-    if type == "application/octet-stream" do
-      String.downcase(Enum.join([uuid, ext], "."))
-    else
-      String.downcase(Enum.join([uuid, List.last(String.split(type, "/"))], "."))
+    case type do
+      "application/octet-stream" ->
+        String.downcase(Enum.join([uuid, ext], "."))
+
+      "audio/mpeg" ->
+        String.downcase(Enum.join([uuid, "mp3"], "."))
+
+      _ ->
+        String.downcase(Enum.join([uuid, List.last(String.split(type, "/"))], "."))
     end
   end
 
@@ -105,7 +110,21 @@ defp get_name(file, uuid, type, should_dedupe) do
     if should_dedupe do
       create_name(uuid, List.last(String.split(file.filename, ".")), type)
     else
-      file.filename
+      unless String.contains?(file.filename, ".") do
+        case type do
+          "image/png" -> file.filename <> ".png"
+          "image/jpeg" -> file.filename <> ".jpg"
+          "image/gif" -> file.filename <> ".gif"
+          "video/webm" -> file.filename <> ".webm"
+          "video/mp4" -> file.filename <> ".mp4"
+          "audio/mpeg" -> file.filename <> ".mp3"
+          "audio/ogg" -> file.filename <> ".ogg"
+          "audio/wav" -> file.filename <> ".wav"
+          _ -> file.filename
+        end
+      else
+        file.filename
+      end
     end
   end
 
diff --git a/test/upload_test.exs b/test/upload_test.exs
index 645f10293..09aa5e068 100644
--- a/test/upload_test.exs
+++ b/test/upload_test.exs
@@ -43,5 +43,18 @@ test "fixes incorrect content type" do
       data = Upload.store(file, true)
       assert hd(data["url"])["mediaType"] == "image/jpeg"
     end
+
+    test "adds missing extension" do
+      File.cp!("test/fixtures/image.jpg", "test/fixtures/image_tmp.jpg")
+
+      file = %Plug.Upload{
+        content_type: "image/jpg",
+        path: Path.absname("test/fixtures/image_tmp.jpg"),
+        filename: "an [image"
+      }
+
+      data = Upload.store(file, false)
+      assert data["name"] == "an [image.jpg"
+    end
   end
 end