This is an automated email from the ASF dual-hosted git repository.

stevel pushed a commit to branch feature-HADOOP-18073-s3a-sdk-upgrade
in repository https://gitbox.apache.org/repos/asf/hadoop.git

commit fb6d12a9a59520e35049783851b5b2f626952d30
Author: Ahmar Suhail <ahma...@amazon.co.uk>
AuthorDate: Wed Jan 18 15:55:26 2023 +0000

    adds in unit check for bytes
---
 .../apache/hadoop/fs/s3a/audit/impl/LoggingAuditor.java    | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git 
a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/audit/impl/LoggingAuditor.java
 
b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/audit/impl/LoggingAuditor.java
index 4005179d201..b48519907b2 100644
--- 
a/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/audit/impl/LoggingAuditor.java
+++ 
b/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3a/audit/impl/LoggingAuditor.java
@@ -265,12 +265,14 @@ public class LoggingAuditor
     private void attachRangeFromRequest(SdkHttpRequest request,
         ExecutionAttributes executionAttributes) {
 
-      if 
(executionAttributes.getAttribute(AwsExecutionAttribute.OPERATION_NAME).equals("GetObject"))
 {
-        if (request.headers() != null
-            && request.headers().get("Range") != null) {
-          String rangeValue = request.headers().get("Range").get(0);
-          String rangeHeader = rangeValue.split("=")[1];
-          referrer.set(AuditConstants.PARAM_RANGE, rangeHeader);
+      if 
(executionAttributes.getAttribute(AwsExecutionAttribute.OPERATION_NAME)
+          .equals("GetObject")) {
+        if (request.headers() != null && request.headers().get("Range") != 
null) {
+          String[] rangeHeader = 
request.headers().get("Range").get(0).split("=");
+          // only set header if range unit is  bytes
+          if (rangeHeader[0].equals("bytes")) {
+            referrer.set(AuditConstants.PARAM_RANGE, rangeHeader[1]);
+          }
         }
       }
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-commits-h...@hadoop.apache.org

Reply via email to