Merge 4.4.0 release to main

CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 4.4.0 — 2024-03-09
+
+### Breaking 
+
+* Drop support of unmaintainable Ruby versions < 3.x
+
+### Fixed
+
+* Update locations where Dry::Monads structure has changed.
+* Sign URL uploads if configured (#139)
+* Start returning proper error message when raising RequestError in poll_upload_response, to hint to users what is going on. Fixes #141
+* When polling, raise if an error is returned (#142)
+* Fix documentation about original file url on simple file upload.
+
+### Changed
+* Support params in Rest client and in file info method, to allow passing custom params like "include=appdata" in `Uploadcare::File.file` calls. Closes #132
+
+
 ## 4.3.6 — 2023-11-18
 
 ### Fixed

lib/uploadcare.rb

@@ -6,6 +6,7 @@
 # Exceptions
 require 'exception/throttle_error'
 require 'exception/request_error'
+require 'exception/retry_error'
 
 # Entities
 require 'entity/entity'

lib/uploadcare/client/uploader_client.rb

@@ -3,6 +3,7 @@
 require_relative 'upload_client'
 require 'retries'
 require 'param/upload/upload_params_generator'
+require 'param/upload/signature_generator'
 
 module Uploadcare
   module Client
@@ -75,15 +76,22 @@ def post(args = {})
       def poll_upload_response(token)
         with_retries(max_tries: Uploadcare.config.max_request_tries,
                      base_sleep_seconds: Uploadcare.config.base_request_sleep,
-                     max_sleep_seconds: Uploadcare.config.max_request_sleep) do
+                     max_sleep_seconds: Uploadcare.config.max_request_sleep,
+                     rescue: RetryError) do
           response = get_upload_from_url_status(token)
+          handle_polling_response(response)
+        end
+      end
 
-          if %w[progress waiting unknown].include?(response.success[:status])
-            raise RequestError, 'Upload is taking longer than expected. Try increasing the max_request_tries config if you know your file uploads will take more time.' # rubocop:disable Layout/LineLength
-          end
-
-          response
+      def handle_polling_response(response)
+        case response.success[:status]
+        when 'error'
+          raise RequestError, response.success[:error]
+        when 'progress', 'waiting', 'unknown'
+          raise RetryError, response.success[:error] || 'Upload is taking longer than expected. Try increasing the max_request_tries config if you know your file uploads will take more time.' # rubocop:disable Layout/LineLength
         end
+
+        response
       end
 
       # Prepares body for upload_many method
@@ -99,13 +107,13 @@ def upload_many_body(arr, options = {})
 
       # Prepare upload_from_url initial request body
       def upload_from_url_body(url, options = {})
-        HTTP::FormData::Multipart.new(
-          options.merge(
-            'pub_key' => Uploadcare.config.public_key,
-            'source_url' => url,
-            'store' => store_value(options[:store])
-          )
-        )
+        opts = {
+          'pub_key' => Uploadcare.config.public_key,
+          'source_url' => url,
+          'store' => store_value(options[:store])
+        }
+        opts.merge!(Param::Upload::SignatureGenerator.call) if Uploadcare.config.sign_uploads
+        HTTP::FormData::Multipart.new(options.merge(opts))
       end
 
       def store_value(store)

lib/uploadcare/exception/request_error.rb

@@ -3,7 +3,6 @@
 module Uploadcare
   module Exception
     # Standard error for invalid API responses
-    class RequestError < StandardError
-    end
+    class RequestError < StandardError; end
   end
 end

lib/uploadcare/exception/retry_error.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module Uploadcare
+  module Exception
+    # Standard error to raise when needing to retry a request
+    class RetryError < StandardError; end
+  end
+end

lib/uploadcare/ruby/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Uploadcare
-  VERSION = '4.3.6'
+  VERSION = '4.4.0'
 end

spec/uploadcare/client/uploader_client_spec.rb

@@ -5,11 +5,12 @@
 module Uploadcare
   module Client
     RSpec.describe UploaderClient do
-      subject { UploaderClient.new }
-      let!(:file) { ::File.open('spec/fixtures/kitten.jpeg') }
-      let!(:another_file) { ::File.open('spec/fixtures/another_kitten.jpeg') }
+      subject { described_class.new }
 
       describe 'upload' do
+        let(:file) { ::File.open('spec/fixtures/kitten.jpeg') }
+        let(:another_file) { ::File.open('spec/fixtures/another_kitten.jpeg') }
+
         it 'uploads a file' do
           VCR.use_cassette('upload_upload') do
             response = subject.upload(file, metadata: { subsystem: 'test' })