IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-25 10:20:58 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: upload URLs from S3 on subfolder installs

Browse Source
This commit is contained in:
Neil Lalonde 2018-08-21 14:58:42 -04:00
parent a00d368d0f
commit 357df2ff4f
2 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/file_store/s3_store.rb
View File

@ -91,7 +91,7 @@ module FileStore
return url if SiteSetting.Upload.s3_cdn_url.blank?
schema = url[/^(https?:)?\/\//, 1]
folder = @s3_helper.s3_bucket_folder_path.nil? ? "" : "#{@s3_helper.s3_bucket_folder_path}/"
url.sub("#{schema}#{absolute_base_url}/#{folder}", "#{SiteSetting.Upload.s3_cdn_url}/")
url.sub("#{schema}#{absolute_base_url}/#{folder}", "#{SiteSetting.Upload.s3_cdn_url}#{Discourse.base_uri}/")
end
def cache_avatar(avatar, user_id)

17
spec/components/file_store/s3_store_spec.rb
View File

@ -282,4 +282,21 @@ describe FileStore::S3Store do
assert_path("https://hello", nil)
end
end
describe '.cdn_url' do
it 'uses the correct path' do
url = "//s3-upload-bucket.s3-us-west-2.amazonaws.com/livechat/original/1X/2252ae522257fc537351e47bbdd34698936b6c38.jpeg"
expect(store.cdn_url(url)).to end_with('/livechat/original/1X/2252ae522257fc537351e47bbdd34698936b6c38.jpeg')
end
it 'supports subfolder' do
SiteSetting.s3_upload_bucket = 's3-upload-bucket/livechat/community'
GlobalSetting.stubs(:relative_url_root).returns('/community')
Discourse.stubs(:base_uri).returns("/community")
url = "//s3-upload-bucket.s3-us-west-2.amazonaws.com/livechat/community/original/1X/2252ae522257fc537351e47bbdd34698936b6c38.jpeg"
expect(store.cdn_url(url)).to end_with('/livechat/community/original/1X/2252ae522257fc537351e47bbdd34698936b6c38.jpeg')
end
end
end