IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-28 19:53:53 -06:00
Code Issues Packages Projects Releases Wiki Activity

Revert "FIX: upload URLs from S3 on subfolder installs"

Browse Source 
This reverts commit 357df2ff4f.
This commit is contained in:
Sam 2018-08-22 10:51:40 +10:00
parent bed34b52b5
commit 26c96e97e5
2 changed files with 1 additions and 18 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}#{Discourse.base_uri}/")
url.sub("#{schema}#{absolute_base_url}/#{folder}", "#{SiteSetting.Upload.s3_cdn_url}/")
end
def cache_avatar(avatar, user_id)

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

@ -282,21 +282,4 @@ 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