diff options
| author | Roger Braun <roger@rogerbraun.net> | 2017-08-09 16:45:05 +0200 |
|---|---|---|
| committer | Roger Braun <roger@rogerbraun.net> | 2017-08-09 16:45:05 +0200 |
| commit | a32784fe6d2237d01835db5d70fa27bc52bdbb0e (patch) | |
| tree | 248da338864a9c9d6e10c0720f3c2855582094d2 | |
| parent | 9c35456d77cec6e37c18b78a20719a7dac865532 (diff) | |
| download | pleroma-a32784fe6d2237d01835db5d70fa27bc52bdbb0e.tar.gz | |
Cowboy and Elixir don't seem to agree on uri encoding...
| -rw-r--r-- | lib/pleroma/upload.ex | 4 | ||||
| -rw-r--r-- | test/upload_test.exs | 6 |
2 files changed, 5 insertions, 5 deletions
diff --git a/lib/pleroma/upload.ex b/lib/pleroma/upload.ex index 5691c6845..2717377a3 100644 --- a/lib/pleroma/upload.ex +++ b/lib/pleroma/upload.ex @@ -13,7 +13,7 @@ defmodule Pleroma.Upload do "url" => [%{ "type" => "Link", "mediaType" => file.content_type, - "href" => url_for(Path.join(uuid, URI.encode(file.filename))) + "href" => url_for(Path.join(uuid, :cow_uri.urlencode(file.filename))) }], "name" => file.filename, "uuid" => uuid @@ -38,7 +38,7 @@ defmodule Pleroma.Upload do "url" => [%{ "type" => "Link", "mediaType" => content_type, - "href" => url_for(Path.join(uuid, URI.encode(filename))) + "href" => url_for(Path.join(uuid, :cow_uri.urlencode(filename))) }], "name" => filename, "uuid" => uuid diff --git a/test/upload_test.exs b/test/upload_test.exs index d29a1f2ba..71041e83c 100644 --- a/test/upload_test.exs +++ b/test/upload_test.exs @@ -4,10 +4,10 @@ defmodule Pleroma.UploadTest do describe "Storing a file" do test "copies the file to the configured folder" do - file = %Plug.Upload{content_type: "image/jpg", path: Path.absname("test/fixtures/image.jpg"), filename: "an image.jpg"} + file = %Plug.Upload{content_type: "image/jpg", path: Path.absname("test/fixtures/image.jpg"), filename: "an [image.jpg"} data = Upload.store(file) - assert data["name"] == "an image.jpg" - assert List.first(data["url"])["href"] == "http://localhost:4001/media/#{data["uuid"]}/an%20image.jpg" + assert data["name"] == "an [image.jpg" + assert List.first(data["url"])["href"] == "http://localhost:4001/media/#{data["uuid"]}/an%20%5Bimage.jpg" end end end |