Add `S3_DISABLE_CHECKSUM_MODE` environment variable for compatibility with some S3-compatible providers (#26435)
parent
3dd3c50811
commit
4bc0dd751c
|
@ -90,13 +90,19 @@ if ENV['S3_ENABLED'] == 'true'
|
||||||
|
|
||||||
# Some S3-compatible providers might not actually be compatible with some APIs
|
# Some S3-compatible providers might not actually be compatible with some APIs
|
||||||
# used by kt-paperclip, see https://github.com/mastodon/mastodon/issues/16822
|
# used by kt-paperclip, see https://github.com/mastodon/mastodon/issues/16822
|
||||||
if ENV['S3_FORCE_SINGLE_REQUEST'] == 'true'
|
# and https://github.com/mastodon/mastodon/issues/26394
|
||||||
|
if ENV['S3_FORCE_SINGLE_REQUEST'] == 'true' || ENV['S3_DISABLE_CHECKSUM_MODE'] == 'true'
|
||||||
module Paperclip
|
module Paperclip
|
||||||
module Storage
|
module Storage
|
||||||
module S3Extensions
|
module S3Extensions
|
||||||
def copy_to_local_file(style, local_dest_path)
|
def copy_to_local_file(style, local_dest_path)
|
||||||
log("copying #{path(style)} to local file #{local_dest_path}")
|
log("copying #{path(style)} to local file #{local_dest_path}")
|
||||||
s3_object(style).download_file(local_dest_path, { mode: 'single_request' })
|
|
||||||
|
options = {}
|
||||||
|
options[:mode] = 'single_request' if ENV['S3_FORCE_SINGLE_REQUEST'] == 'true'
|
||||||
|
options[:checksum_mode] = 'DISABLED' if ENV['S3_DISABLE_CHECKSUM_MODE'] == 'true'
|
||||||
|
|
||||||
|
s3_object(style).download_file(local_dest_path, options)
|
||||||
rescue Aws::Errors::ServiceError => e
|
rescue Aws::Errors::ServiceError => e
|
||||||
warn("#{e} - cannot copy #{path(style)} to local file #{local_dest_path}")
|
warn("#{e} - cannot copy #{path(style)} to local file #{local_dest_path}")
|
||||||
false
|
false
|
||||||
|
|
Loading…
Reference in New Issue