Merge pull request #104 from NipunaRanasinghe/2201.4.x

[2201.4.x] Fix object retrieval for partial content
ballerina-platform · Feb 15, 2024 · 3adedda · 3adedda
2 parents cc7214d + 6db2c90
commit 3adedda
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/s3/Ballerina.toml b/s3/Ballerina.toml
@@ -2,7 +2,7 @@
 distribution = "2201.4.1"
 org = "ballerinax"
 name = "aws.s3"
-version = "3.2.0"
+version = "3.2.1"
 license= ["Apache-2.0"]
 authors = ["Ballerina"]
 keywords = ["Content & Files/File Management & Storage", "Cost/Paid", "Vendor/Amazon"]

diff --git a/s3/Dependencies.toml b/s3/Dependencies.toml
@@ -41,7 +41,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "crypto"
-version = "2.3.0"
+version = "2.3.2"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"},
 	{org = "ballerina", name = "time"}
@@ -66,7 +66,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "http"
-version = "2.6.1"
+version = "2.6.2"
 dependencies = [
 	{org = "ballerina", name = "auth"},
 	{org = "ballerina", name = "cache"},
@@ -297,7 +297,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "time"
-version = "2.2.4"
+version = "2.2.5"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"}
 ]
@@ -308,7 +308,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "url"
-version = "2.2.3"
+version = "2.2.4"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"}
 ]
@@ -319,7 +319,7 @@ modules = [
 [[package]]
 org = "ballerinax"
 name = "aws.s3"
-version = "3.2.0"
+version = "3.2.1"
 dependencies = [
 	{org = "ballerina", name = "crypto"},
 	{org = "ballerina", name = "http"},

diff --git a/s3/client.bal b/s3/client.bal
@@ -165,7 +165,7 @@ public isolated client class Client {
         check generateSignature(self.accessKeyId, self.secretAccessKey, self.region, GET, requestURI, UNSIGNED_PAYLOAD,
             requestHeaders);
         http:Response httpResponse = check self.amazonS3->get(requestURI, requestHeaders);
-        if (httpResponse.statusCode == http:STATUS_OK) {
+        if (httpResponse.statusCode == http:STATUS_OK || (httpResponse.statusCode == http:STATUS_PARTIAL_CONTENT && objectRetrievalHeaders?.range != ())) {
             if byteArraySize is int {
                 return httpResponse.getByteStream(byteArraySize);
             }