scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2025-11-19 00:08:49 +08:00
Code Issues Projects Releases Packages Wiki Activity

Fix API endpoints that return file URLs [SCI-2771]

Browse source
This commit is contained in:
Oleksii Kriuchykhin 2018-10-28 18:39:34 +01:00
parent c8d381a0a2
commit d1e6c64d5e
2 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/serializers/api/v1/repository_asset_value_serializer.rb
View file

@ -18,7 +18,7 @@ module Api
end
def url
if !object.asset&.file_present
if !object.asset&.file&.exists?
nil
elsif object.asset&.file&.is_stored_on_s3?
object.asset.presigned_url(download: true)

6
app/serializers/api/v1/result_asset_serializer.rb
View file

@ -19,14 +19,12 @@ module Api
end
def url
if !object.asset&.file_present
if !object.asset&.file&.exists?
nil
elsif object.asset&.file&.is_stored_on_s3?
object.asset.presigned_url(download: true)
else
# TODO
# separate api endpoint for local files download is needed
'url'#download_asset_path(object.asset.id)
object.asset.file.url
end
end
end