aws-sdk-s3 1.233.1 → 1.233.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a94f968574a4140131a94ed10f6e238db29a2cb3ceac5eaab1bccdda34efa4f
-  data.tar.gz: 63bbf8c2512e03438f6e22c634bb9070cc52e0b06096d7a6945dc55100303631
+  metadata.gz: 7dd065c16e48d0c0f7d227b772a6d6716478341cc697d4b552ba6098a6cfab06
+  data.tar.gz: b3322ba1c97612a0796f149b92c22d232f385480d7be7bd091125c1818a8f0c5
 SHA512:
-  metadata.gz: 46c2a374e3f4bbd446d32b3403f08e2b648f37dc5f1cfb1d1c2224a211b43e6450ebdaa9cc3bbf9c6fa3e2a5b0884671127baf13163391fb11647a339f34ae5f
-  data.tar.gz: 564b1d9d422faa9180efc8e86f31803c517335608ff98eb05fa328094cc5de1a23cd22f13afeb09aa3e84aeb5f0e0f3b8a2e9a2f69437403efcbc3aa7bb2ebb2
+  metadata.gz: 2591b76c9a1118cc31cfd6c85ddbf29767ad50ac197349f93f1a33ae31b0138f74abf455aef08ab2f8f12f75337b8bc5daba1bf858657a3f93fe2f8f87818f85
+  data.tar.gz: 232fa0a2eaf103bf1738acea96860f9f59657f13c1507fa14f4dc469fadf052de744e58ff9f62648bbb1d9779e0913919731d114ee71ba503d8b0afe014ff177

data/CHANGELOG.md

@@ -1,6 +1,13 @@
 Unreleased Changes
 ------------------
 
+1.233.2 (2026-10-06)
+------------------
+
+* Issue - Fix `download_file` on `TransferManager` and `Aws::S3::Object` leaving a corrupt file at the destination when a part raises a non-`StandardError`, and leaving a temp file behind when a caller-provided executor rejects a part.
+
+* Issue - Prevent `upload_directory` and `download_directory` on `TransferManager` from hanging when listing files or objects raises a non-`StandardError`, such as from a `:filter_callback`.
+
 1.233.1 (2026-10-01)
 ------------------
 

data/VERSION

@@ -1 +1 @@
-1.233.1
+1.233.2

data/lib/aws-sdk-s3/client.rb

@@ -24209,7 +24209,7 @@ module Aws::S3
         tracer: tracer
       )
       context[:gem_name] = 'aws-sdk-s3'
-      context[:gem_version] = '1.233.1'
+      context[:gem_version] = '1.233.2'
       Seahorse::Client::Request.new(handlers, context)
     end
 

data/lib/aws-sdk-s3/directory_downloader.rb

@@ -148,8 +148,9 @@ module Aws
             stream_objects
             @object_queue << DONE_MARKER
           rescue ClosedQueueError
-            # abort requested
-          rescue StandardError => e
+            nil # abort requested
+          # Any error, or the consumer waits on the queue forever
+          rescue Exception => e # rubocop:disable Lint/RescueException
             close
             raise e
           end

data/lib/aws-sdk-s3/directory_uploader.rb

@@ -156,6 +156,10 @@ module Aws
             # encountered a traversal error, we must abort immediately
             close
             raise DirectoryUploadError, "Directory traversal failed for '#{@source_dir}': #{e.message}"
+          # Any other error, or the consumer waits on the queue forever
+          rescue Exception => e # rubocop:disable Lint/RescueException
+            close
+            raise e
           end
 
           while (file = @file_queue.shift) && file != DONE_MARKER

data/lib/aws-sdk-s3/file_downloader.rb

@@ -60,7 +60,7 @@ module Aws
       end
 
       def download_with_executor(part_list, total_size, opts)
-        download_attempts = 0
+        queued_parts = 0
         completion_queue = Queue.new
         abort_download = false
         error = nil
@@ -69,24 +69,31 @@ module Aws
         while (part = part_list.shift)
           break if abort_download
 
-          download_attempts += 1
-          @executor.post(part) do |p|
-            update_progress(progress, p)
-            resp = @client.get_object(p.params)
-            range = extract_range(resp.content_range)
-            validate_range(range, p.params[:range]) if p.params[:range]
-            write(resp.body, range, opts)
-
-            execute_checksum_callback(resp, opts)
+          begin
+            @executor.post(part) do |p|
+              update_progress(progress, p)
+              resp = @client.get_object(p.params)
+              range = extract_range(resp.content_range)
+              validate_range(range, p.params[:range]) if p.params[:range]
+              write(resp.body, range, opts)
+
+              execute_checksum_callback(resp, opts)
+            # Any error, or a corrupt file replaces the destination
+            rescue Exception => e # rubocop:disable Lint/RescueException
+              abort_download = true
+              error = e
+            ensure
+              completion_queue << :done
+            end
+            queued_parts += 1
           rescue StandardError => e
-            abort_download = true
+            # Rejected by the executor, wait for queued parts so none write after cleanup
             error = e
-          ensure
-            completion_queue << :done
+            break
           end
         end
 
-        download_attempts.times { completion_queue.pop }
+        queued_parts.times { completion_queue.pop }
         raise error unless error.nil?
       end
 

data/lib/aws-sdk-s3/multipart_file_uploader.rb

@@ -169,7 +169,6 @@ module Aws
             queued_parts += 1
           rescue StandardError => e
             # Rejected by the executor; abort rather than orphan the upload.
-            abort_upload = true
             errors << e
             break
           end

data/lib/aws-sdk-s3.rb

@@ -75,7 +75,7 @@ module Aws::S3
   autoload :ObjectVersion, 'aws-sdk-s3/object_version'
   autoload :EventStreams, 'aws-sdk-s3/event_streams'
 
-  GEM_VERSION = '1.233.1'
+  GEM_VERSION = '1.233.2'
 
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aws-sdk-s3
 version: !ruby/object:Gem::Version
-  version: 1.233.1
+  version: 1.233.2
 platform: ruby
 authors:
 - Amazon Web Services