aws-sdk-s3 1.123.0 → 1.123.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0bce7e3809967ffcc0162974361501c6d816e367767032d31c4fa1dd90ca7542
-  data.tar.gz: 9ec29e833cbd5b4506da1b40b070a764b4925c056414bac68a33b008bbe1ab0a
+  metadata.gz: 18c5a3063c80c67d11fcb593d262d3eb15b88edf2c25abdaaff2f12c57620274
+  data.tar.gz: b5a6e05f048778f72ca6f34947b4e7b4ed318a447d271764df129dc9e7bcdda3
 SHA512:
-  metadata.gz: f3d113f0edcb793cb22503c4ddcfebd563b7b141652ae70a6471e4bb96e0906453767806bf8a36343f9732f4be1d3070eac1c901222cb3246b82c956ddc8179c
-  data.tar.gz: 918bc83ab79e362b3cfd7ec4ee3a4d701b6c59ca0474811fe1011d9e2b100cc6c8bd583f5fcf887ca8311dcb6c6365ffed851acd06f3eec99732b210e78e8e3b
+  metadata.gz: f09b9adb0fcb1c14e974c482737bce5017377c9b7771f61e3330681a1094a42055edc2727f934e70d1097474163cc98bc2042a2a6880135c2bf9ac97ba5bb88f
+  data.tar.gz: c702479639bb64d638874d05477aa34b3581f3560f573ebd5acaea3d5e272bafd27ba3a491260ddb6697b450a5efaaf240c6699d2c87072b4ccc2f75b6ab00dc

data/CHANGELOG.md

@@ -1,6 +1,11 @@
 Unreleased Changes
 ------------------
 
+1.123.1 (2023-06-02)
+------------------
+
+* Issue - Fix multipart `download_file` so that it does not download bytes out of range (#2859).
+
 1.123.0 (2023-05-31)
 ------------------
 

data/VERSION

@@ -1 +1 @@
-1.123.0
+1.123.1

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

@@ -15601,7 +15601,7 @@ module Aws::S3
         params: params,
         config: config)
       context[:gem_name] = 'aws-sdk-s3'
-      context[:gem_version] = '1.123.0'
+      context[:gem_version] = '1.123.1'
       Seahorse::Client::Request.new(handlers, context)
     end
 

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

@@ -58,15 +58,19 @@ module Aws
         resp = @client.head_object(@params.merge(part_number: 1))
         count = resp.parts_count
         if count.nil? || count <= 1
-          resp.content_length < MIN_CHUNK_SIZE ?
-            single_request :
+          if resp.content_length <= MIN_CHUNK_SIZE
+            single_request
+          else
             multithreaded_get_by_ranges(construct_chunks(resp.content_length))
+          end
         else
           # partNumber is an option
           resp = @client.head_object(@params)
-          resp.content_length < MIN_CHUNK_SIZE ?
-            single_request :
+          if resp.content_length <= MIN_CHUNK_SIZE
+            single_request
+          else
             compute_mode(resp.content_length, count)
+          end
         end
       end
 
@@ -84,10 +88,11 @@ module Aws
         offset = 0
         default_chunk_size = compute_chunk(file_size)
         chunks = []
-        while offset <= file_size
+        while offset < file_size
           progress = offset + default_chunk_size
-          chunks << "bytes=#{offset}-#{progress < file_size ? progress : file_size}"
-          offset = progress + 1
+          progress = file_size if progress > file_size
+          chunks << "bytes=#{offset}-#{progress - 1}"
+          offset = progress
         end
         chunks
       end
@@ -96,12 +101,9 @@ module Aws
         if @chunk_size && @chunk_size > file_size
           raise ArgumentError, ":chunk_size shouldn't exceed total file size."
         else
-          chunk_size = @chunk_size || [
-            (file_size.to_f / MAX_PARTS).ceil,
-            MIN_CHUNK_SIZE
+          @chunk_size || [
+            (file_size.to_f / MAX_PARTS).ceil, MIN_CHUNK_SIZE
           ].max.to_i
-          chunk_size -= 1 if file_size % chunk_size == 1
-          chunk_size
         end
       end
 

data/lib/aws-sdk-s3.rb

@@ -73,6 +73,6 @@ require_relative 'aws-sdk-s3/event_streams'
 # @!group service
 module Aws::S3
 
-  GEM_VERSION = '1.123.0'
+  GEM_VERSION = '1.123.1'
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: aws-sdk-s3
 version: !ruby/object:Gem::Version
-  version: 1.123.0
+  version: 1.123.1
 platform: ruby
 authors:
 - Amazon Web Services
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-05-31 00:00:00.000000000 Z
+date: 2023-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-kms