FIX: add short_path to upload_serializer (#9417)

FIX: add short_path to upload_serializer (#9417)

What problem I am trying to solve?

When an encrypted message is crafted and the image is added - discourse needs a hard refresh to display that image.

What is happening?

Everything starts here - when the upload is finished we add serialized object to the cache

Then, discourse-encrypt is trying to get an image from the cache and use short_path property

Why is it working after a hard refresh?

After refresh, we populate cache once again using that function:

And lookup_urls method from backend is returning short_path

TL;DR We should expose short path in upload serializer. I ensured that this serializer is used only when attachments are uploaded so it should not affect performance.

diff --git a/app/serializers/upload_serializer.rb b/app/serializers/upload_serializer.rb
index 938ab5b..a04300b 100644
--- a/app/serializers/upload_serializer.rb
+++ b/app/serializers/upload_serializer.rb
@@ -11,6 +11,7 @@ class UploadSerializer < ApplicationSerializer
+             :short_path,
diff --git a/spec/serializers/upload_serializer_spec.rb b/spec/serializers/upload_serializer_spec.rb
index 3865000..b0f5e64 100644
--- a/spec/serializers/upload_serializer_spec.rb
+++ b/spec/serializers/upload_serializer_spec.rb
@@ -14,6 +14,7 @@ RSpec.describe UploadSerializer do
     expect(json_data['height']).to eql upload.height
     expect(json_data['thumbnail_width']).to eql upload.thumbnail_width
     expect(json_data['thumbnail_height']).to eql upload.thumbnail_height
+    expect(json_data['short_path']).to eql upload.short_path
   context "when the upload is secure" do

GitHub sha: c21df228

This commit appears in #9417 which was approved by eviltrout. It was merged by lis2.