Upload: Fix uploading with a ? in the filename

diff --git a/lib/pleroma/upload.ex b/lib/pleroma/upload.ex
index 185ba25fa15effd4b85e98c675d9be4823bb0a26..1d8b073afc5db34f88865becefc38b6611fa7487 100644 (file)
--- a/lib/pleroma/upload.ex
+++ b/lib/pleroma/upload.ex
@@ -215,7 +215,12 @@ defmodule Pleroma.Upload do
   end
 
   defp url_from_spec(base_url, {:file, path}) do
-    [base_url, "media", URI.encode(path)]
+    path =
+      path
+      |> URI.encode()
+      |> String.replace("?", "%3F")
+
+    [base_url, "media", path]
     |> Path.join()
   end
 

diff --git a/test/upload_test.exs b/test/upload_test.exs
index bda5033616261b6d11747a1c700e80f7e4641a16..ffef74270a1cf71f0784d302f816589ebc0be3c4 100644 (file)
--- a/test/upload_test.exs
+++ b/test/upload_test.exs
@@ -152,5 +152,20 @@ defmodule Pleroma.UploadTest do
 
       assert Path.basename(attachment_url["href"]) == "an%E2%80%A6%20image.jpg"
     end
+
+    test "replaces ? (question-mark) to %3f" 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.jpg"
+      }
+
+      {:ok, data} = Upload.store(file)
+      [attachment_url | _] = data["url"]
+
+      assert Path.basename(attachment_url["href"]) == "an%3Fimage.jpg"
+    end
   end
 end