Merge pull request #1400 from okriuchykhin/ok_SCI_2727

Make AWS access key variables optional [SCI-2727]
This commit is contained in:
Alex Kriuchykhin 2018-11-28 14:45:45 +01:00 committed by GitHub
commit a4b75f00d2
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 11 deletions

View file

@ -1,10 +1,5 @@
if ENV['AWS_ACCESS_KEY_ID']
Aws.config.update(
region: ENV['AWS_REGION'],
credentials: Aws::Credentials.new(
ENV['AWS_ACCESS_KEY_ID'], ENV['AWS_SECRET_ACCESS_KEY']
)
)
# frozen_string_literal: true
if ENV['PAPERCLIP_STORAGE'] == 's3'
S3_BUCKET = Aws::S3::Resource.new.bucket(ENV['S3_BUCKET'])
end

View file

@ -14,10 +14,8 @@ Paperclip::Attachment.default_options.merge!(
Paperclip::UriAdapter.register
if ENV['PAPERCLIP_STORAGE'] == 's3'
if ENV['S3_BUCKET'].nil? || ENV['AWS_REGION'].nil? ||
ENV['AWS_ACCESS_KEY_ID'].nil? || ENV['AWS_SECRET_ACCESS_KEY'].nil?
puts 'WARNING! Environment variables S3_BUCKET, AWS_REGION, '\
'AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY must be set.'
if ENV['S3_BUCKET'].nil? || ENV['AWS_REGION'].nil?
puts 'WARNING! Environment variables S3_BUCKET and AWS_REGION must be set.'
exit 1
end