[ 
https://issues.apache.org/jira/browse/HADOOP-19736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18034392#comment-18034392
 ] 

ASF GitHub Bot commented on HADOOP-19736:
-----------------------------------------

anmolanmol1234 commented on code in PR #8051:
URL: https://github.com/apache/hadoop/pull/8051#discussion_r2481307275


##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/utils/DelegationSASGenerator.java:
##########
@@ -117,6 +125,15 @@ public String getDelegationSAS(String accountName, String 
containerName, String
     qb.addQuery("ske", ske);
     qb.addQuery("sks", sks);
     qb.addQuery("skv", skv);
+
+    //skdutid and sduoid are required for user bound SAS only
+    if(!Objects.equals(skdutid, EMPTY_STRING)){

Review Comment:
   spaces after if





> ABFS: Support for new auth type: User-bound SAS
> -----------------------------------------------
>
>                 Key: HADOOP-19736
>                 URL: https://issues.apache.org/jira/browse/HADOOP-19736
>             Project: Hadoop Common
>          Issue Type: Task
>          Components: fs/azure
>    Affects Versions: 3.4.1, 3.4.2
>            Reporter: Manika Joshi
>            Assignee: Manika Joshi
>            Priority: Major
>              Labels: pull-request-available
>
> Adding support for new authentication type: user bound SAS



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to