carrierwave-aws 1.5.0 → 1.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b420ef988503bad151d75b1794b45ef1d3c2906077df034ddbdf067369f003aa
-  data.tar.gz: 167626ab412b4b49d26927c7044445974120a273c7fda2398faea3f3d549ff0c
+  metadata.gz: a5f21a2f65d6fc529886cc06a0b9a796d61775b249655519c0401336b0076383
+  data.tar.gz: 7c33f49c78db4c5996e6db75e8841864184db6e9a3c70186a982735f59c5c253
 SHA512:
-  metadata.gz: 0f1c69cf12347fa87723f671de01166a1b6373e3863b36acca438f6cc253d1e50cd76f6835c55d860078003d08d9d9692b596d40736f7eade764a458de5cb826
-  data.tar.gz: c153eaa6992b88e00c34d3eeca7e8666b6f5f9a665a137b365214f3dc2be9520ef8701cbd3a4b6451d854d160466b4a4e16b3469171a0eac2540553f28d3fc5f
+  metadata.gz: d5f7d35bfc4eaae8eef5a8c314b175c1e37a0394f19896c5ef96daf38c1d16ae37f269d03cb832f66765cfeba2b05d5339d0b6ea649fc0524e4614fd280268ca
+  data.tar.gz: 7277c6e4455acfc23dc2afd4f2e23d25bd5e550606c395151de2a75c969874cfd8b77b492d631c266c28f7f737d3c2a4cd07c679a1a12e404fb3022ec8e916da

data/lib/carrierwave/aws/version.rb

@@ -2,6 +2,6 @@
 
 module Carrierwave
   module AWS
-    VERSION = '1.5.0'
+    VERSION = '1.6.1'
   end
 end

data/lib/carrierwave/storage/aws_file.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'active_support/core_ext/module/delegation'
+
 module CarrierWave
   module Storage
     class AWSFile
@@ -55,14 +57,19 @@ module CarrierWave
       def store(new_file)
         if new_file.is_a?(self.class)
           new_file.move_to(path)
+        elsif Aws::S3.const_defined?(:TransferManager)
+          options = aws_options.write_options(new_file).except(:body)
+          options[:multipart_threshold] = AWSOptions::MULTIPART_THRESHOLD
+          Aws::S3::TransferManager.new(client: connection.client).upload_file(new_file.path, bucket: bucket.name,
+                                                                                             key: path, **options)
         else
           file.upload_file(new_file.path, aws_options.write_options(new_file))
         end
       end
 
       def copy_to(new_path)
-        bucket.object(new_path).copy_from(
-          "#{bucket.name}/#{file.key}",
+        file.copy_to(
+          bucket.object(new_path),
           aws_options.copy_options(self)
         )
       end
@@ -79,12 +86,16 @@ module CarrierWave
       end
 
       def authenticated_url(options = {})
+        if asset_host && asset_host == bucket.name
+          # Can't use https, since plain S3 doesn't support custom TLS certificates
+          options = options.reverse_merge(secure: false, virtual_host: true)
+        end
         file.presigned_url(:get, aws_options.expiration_options(options))
       end
 
       def public_url
-        if uploader.asset_host
-          "#{uploader.asset_host}/#{uri_path}"
+        if asset_host
+          "#{asset_host}/#{uri_path}"
         else
           file.public_url.to_s
         end
@@ -117,6 +128,14 @@ module CarrierWave
       def public?
         uploader.aws_acl.to_s == 'public-read' || uploader.asset_host_public
       end
+
+      def asset_host
+        if uploader.asset_host.respond_to? :call
+          uploader.asset_host.call(self)
+        else
+          uploader.asset_host
+        end
+      end
     end
   end
 end

data/lib/carrierwave/storage/aws_options.rb

@@ -3,7 +3,10 @@
 module CarrierWave
   module Storage
     class AWSOptions
-      MULTIPART_TRESHOLD = 15 * 1024 * 1024
+      MULTIPART_THRESHOLD = 15 * 1024 * 1024
+
+      # Backward compatibility
+      MULTIPART_TRESHOLD = MULTIPART_THRESHOLD
 
       attr_reader :uploader
 
@@ -26,7 +29,7 @@ module CarrierWave
       def move_options(file)
         {
           acl: uploader.aws_acl,
-          multipart_copy: file.size >= MULTIPART_TRESHOLD
+          multipart_copy: file.size >= MULTIPART_THRESHOLD
         }.merge(aws_attributes).merge(aws_write_options)
       end
       alias copy_options move_options
@@ -42,6 +45,7 @@ module CarrierWave
       def aws_attributes
         attributes = uploader.aws_attributes
         return {} if attributes.nil?
+
         attributes.respond_to?(:call) ? attributes.call : attributes
       end
 

data/lib/carrierwave-aws.rb

@@ -40,6 +40,8 @@ module CarrierWave
       end
 
       def self.normalized_acl(acl)
+        return nil if acl.nil?
+
         normalized = acl.to_s.downcase.sub('_', '-')
 
         unless ACCEPTED_ACL.include?(normalized)

data/spec/carrierwave/storage/aws_file_spec.rb

@@ -86,7 +86,7 @@ describe CarrierWave::Storage::AWSFile do
       allow(uploader).to receive(:aws_authenticated_url_expiration) { 60 }
       allow(uploader).to receive(:asset_host_public) { false }
 
-      expect(file).to receive(:presigned_url).with(:get, expires_in: 60)
+      expect(file).to receive(:presigned_url).with(:get, { expires_in: 60 })
 
       aws_file.url
     end
@@ -110,6 +110,18 @@ describe CarrierWave::Storage::AWSFile do
 
       expect(aws_file.url).to eq('http://example.com/files/1/file.txt')
     end
+
+    it 'accepts the asset_host given as a proc' do
+      allow(uploader).to receive(:aws_acl) { :'public-read' }
+      allow(uploader).to receive(:asset_host) do
+        proc do |file|
+          expect(file).to be_instance_of(CarrierWave::Storage::AWSFile)
+          'https://example.com'
+        end
+      end
+
+      expect(aws_file.url).to eq('https://example.com/files/1/file.txt')
+    end
   end
 
   describe '#store' do
@@ -129,9 +141,36 @@ describe CarrierWave::Storage::AWSFile do
         CarrierWave::SanitizedFile.new('spec/fixtures/image.png')
       end
 
-      it 'uploads the file using with multipart support' do
-        expect(file).to(receive(:upload_file)
-                              .with(new_file.path, an_instance_of(Hash)))
+      it 'uploads the file using TransferManager' do
+        client = instance_double('Aws::S3::Client')
+
+        allow(bucket).to receive(:name).and_return('example-com')
+        allow(connection).to receive(:client).and_return(client)
+
+        if Aws::S3.const_defined?(:TransferManager)
+          transfer_manager = instance_double('Aws::S3::TransferManager')
+          allow(Aws::S3::TransferManager).to receive(:new).with(client: client).and_return(transfer_manager)
+
+          expect(transfer_manager).to receive(:upload_file).with(
+            new_file.path,
+            bucket: 'example-com',
+            key: path,
+            acl: :'public-read',
+            content_type: new_file.content_type,
+            encryption_key: 'def',
+            multipart_threshold: CarrierWave::Storage::AWSOptions::MULTIPART_THRESHOLD
+          )
+        else
+          expect(file).to receive(:upload_file).with(
+            new_file.path,
+            hash_including(
+              acl: :'public-read',
+              content_type: new_file.content_type,
+              encryption_key: 'def'
+            )
+          )
+        end
+
         aws_file.store(new_file)
       end
     end
@@ -144,4 +183,21 @@ describe CarrierWave::Storage::AWSFile do
       expect(aws_file.public_url).to eq 'http://example.com/uploads/images/jekyll%2Band%2Bhyde.txt'
     end
   end
+
+  describe '#copy_to' do
+    let(:new_s3_object) { instance_double('Aws::S3::Object') }
+
+    it 'copies file to target path' do
+      new_path = 'files/2/file.txt'
+      expect(bucket).to receive(:object).with(new_path).and_return(new_s3_object)
+      expect(file).to receive(:size).at_least(:once).and_return(1024)
+      expect(file).to(
+        receive(:copy_to).with(
+          new_s3_object,
+          aws_file.aws_options.copy_options(aws_file)
+        )
+      )
+      aws_file.copy_to new_path
+    end
+  end
 end

data/spec/carrierwave/storage/aws_options_spec.rb

@@ -39,8 +39,8 @@ describe CarrierWave::Storage::AWSOptions do
       write_options = options.write_options(file)
 
       expect(write_options).to include(
-        acl:            'public-read',
-        content_type:   'image/png',
+        acl: 'public-read',
+        content_type: 'image/png',
         encryption_key: 'def'
       )
       expect(write_options[:body].path).to eq(file.path)
@@ -54,7 +54,7 @@ describe CarrierWave::Storage::AWSOptions do
 
     it 'works if aws_attributes is a Proc' do
       expect(uploader).to receive(:aws_attributes).and_return(
-        -> { { expires: 1.week.from_now.httpdate } }
+        -> { { expires: (Date.today + 7).httpdate } }
       )
 
       expect { options.write_options(file) }.to_not raise_error

data/spec/carrierwave/support/uri_filename_spec.rb

@@ -1,8 +1,6 @@
 require 'spec_helper'
 
 describe CarrierWave::Support::UriFilename do
-  UriFilename = CarrierWave::Support::UriFilename
-
   describe '.filename' do
     it 'extracts a decoded filename from file uri' do
       samples = {
@@ -11,7 +9,7 @@ describe CarrierWave::Support::UriFilename do
       }
 
       samples.each do |uri, name|
-        expect(UriFilename.filename(uri)).to eq(name)
+        expect(CarrierWave::Support::UriFilename.filename(uri)).to eq(name)
       end
     end
   end

data/spec/carrierwave-aws_spec.rb

@@ -20,7 +20,7 @@ describe CarrierWave::Uploader::Base do
   end
 
   describe '#aws_acl' do
-    it 'allows known acess control values' do
+    it 'allows known access control values' do
       expect do
         uploader.aws_acl = 'private'
         uploader.aws_acl = 'public-read'
@@ -28,6 +28,13 @@ describe CarrierWave::Uploader::Base do
       end.not_to raise_exception
     end
 
+    it 'allows nil' do
+      uploader.aws_acl = 'public-read'
+      expect do
+        uploader.aws_acl = nil
+      end.to change { uploader.aws_acl }.from('public-read').to(nil)
+    end
+
     it 'does not allow unknown control values' do
       expect do
         uploader.aws_acl = 'everybody'

data/spec/features/storing_files_spec.rb

@@ -73,7 +73,7 @@ describe 'Storing Files', type: :feature do
     image.close
   end
 
-  it 'gets a url for remote files' do
+  it 'gets a public url for remote files' do
     instance.store!(image)
     instance.retrieve_from_store!('image.png')
 
@@ -84,6 +84,34 @@ describe 'Storing Files', type: :feature do
     instance.file.delete
   end
 
+  it 'gets a private url for remote files' do
+    instance.aws_acl = 'private'
+    instance.store!(image)
+    instance.retrieve_from_store!('image.png')
+
+    expect(instance.url).to include(ENV['S3_BUCKET_NAME'])
+    expect(instance.url).to include(instance.path)
+    expect(instance.url).to include('X-Amz-Signature=')
+
+    image.close
+    instance.file.delete
+  end
+
+  it 'respects asset_host for private urls' do
+    allow(instance).to receive(:asset_host).and_return(ENV['S3_BUCKET_NAME'])
+    instance.aws_acl = 'private'
+    instance.store!(image)
+    instance.retrieve_from_store!('image.png')
+
+    expect(URI.parse(instance.url).scheme).to eq 'http'
+    expect(URI.parse(instance.url).hostname).to eq ENV['S3_BUCKET_NAME']
+    expect(instance.url).to include(instance.path)
+    expect(instance.url).to include('X-Amz-Signature=')
+
+    image.close
+    instance.file.delete
+  end
+
   it 'uploads the cache file to the configured bucket' do
     instance.cache!(image)
     instance.retrieve_from_cache!(instance.cache_name)

data/spec/spec_helper.rb

@@ -49,10 +49,15 @@ RSpec.configure do |config|
       cw_config.aws_acl       = :'public-read'
 
       cw_config.aws_credentials = {
-        access_key_id:     ENV['S3_ACCESS_KEY'],
+        access_key_id: ENV['S3_ACCESS_KEY'],
         secret_access_key: ENV['S3_SECRET_ACCESS_KEY'],
-        region:            ENV['AWS_REGION']
-      }
+        region: ENV['AWS_REGION']
+      }.merge(
+        ENV['CI'] && {
+          endpoint: 'http://127.0.0.1:9000',
+          force_path_style: true
+        } || {}
+      )
     end
   end
 end

metadata

@@ -1,55 +1,75 @@
 --- !ruby/object:Gem::Specification
 name: carrierwave-aws
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.6.1
 platform: ruby
 authors:
 - Parker Selbert
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-01 00:00:00.000000000 Z
+date: 2026-01-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: carrierwave
+  name: aws-sdk-s3
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: aws-sdk-s3
+  name: carrierwave
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4'
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '10.0'
 - !ruby/object:Gem::Dependency
@@ -93,7 +113,7 @@ homepage: https://github.com/sorentwo/carrierwave-aws
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -101,15 +121,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.5.22
+signing_key:
 specification_version: 4
 summary: Native aws-sdk support for S3 in CarrierWave
 test_files: