comment suggestions

Author: manika137

hadoop-tools/hadoop-azure/src/site/markdown/index.md

@@ -652,8 +652,9 @@ To know more about how SAS Authentication works refer to
 
 There are three types of SAS supported by Azure Storage:
 - [User Delegation SAS](https://learn.microsoft.com/en-us/rest/api/storageservices/create-user-delegation-sas):
-  Recommended for use with ABFS Driver with HNS Enabled ADLS Gen2 accounts. It
-  is Identity based SAS that works at blob/directory level)
+  SAS-based authentication works with HNS-enabled ADLS Gen2 accounts
+  (recommended for use with ABFS) and is also supported with non-HNS (FNS) Blob
+  accounts. However, it is **NOT SUPPORTED** with FNS-DFS accounts.
 - [Service SAS](https://learn.microsoft.com/en-us/rest/api/storageservices/create-service-sas): Global and works at container level.
 - [Account SAS](https://learn.microsoft.com/en-us/rest/api/storageservices/create-account-sas): Global and works at account level.
 

hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemDelegationSAS.java

@@ -97,7 +97,7 @@ public ITestAzureBlobFileSystemDelegationSAS() throws Exception {
   public void setup() throws Exception {
     isHNSEnabled = this.getConfiguration().getBoolean(
         TestConfigurationKeys.FS_AZURE_TEST_NAMESPACE_ENABLED_ACCOUNT, false);
-    if(!isHNSEnabled){
+    if (!isHNSEnabled) {
       assumeBlobServiceType();
     }
     createFilesystemForSASTests();
@@ -308,6 +308,7 @@ public void testList() throws Exception {
     final AzureBlobFileSystem fs = getFileSystem();
     Path dirPath = new Path(UUID.randomUUID().toString());
     Path filePath = new Path(dirPath, UUID.randomUUID().toString());
+    Path filePath2 = new Path(dirPath, UUID.randomUUID().toString());
 
     fs.mkdirs(dirPath);
 
@@ -316,6 +317,11 @@ public void testList() throws Exception {
       stream.writeBytes("hello");
     }
 
+    // create file with content "bye"
+    try (FSDataOutputStream stream = fs.create(filePath2)) {
+      stream.writeBytes("bye");
+    }
+
     fs.listStatus(filePath);
     fs.listStatus(dirPath);
     fs.listStatus(new Path("/"));

hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/utils/DelegationSASGenerator.java

@@ -24,6 +24,7 @@
 import org.apache.hadoop.fs.azurebfs.extensions.SASTokenProvider;
 import org.apache.hadoop.fs.azurebfs.services.AbfsUriQueryBuilder;
 
+import static org.apache.hadoop.fs.azurebfs.constants.AbfsHttpConstants.ROOT_PATH;
 
 /**
  * Test Delegation SAS generator.
@@ -70,7 +71,7 @@ public String getDelegationSAS(String accountName, String containerName, String
       case SASTokenProvider.DELETE_RECURSIVE_OPERATION:
         sp = "d";
         sr = "d";
-        sdd = path.equals("/")? "0": Integer.toString(StringUtils.countMatches(path, "/"));
+        sdd = path.equals(ROOT_PATH)? "0": Integer.toString(StringUtils.countMatches(path, "/"));
         break;
       case SASTokenProvider.CHECK_ACCESS_OPERATION:
       case SASTokenProvider.GET_ACL_OPERATION:
@@ -84,7 +85,7 @@ public String getDelegationSAS(String accountName, String containerName, String
       case SASTokenProvider.LIST_OPERATION:
         sp = "l";
         sr = "d";
-        sdd = path.equals("/")? "0": Integer.toString(StringUtils.countMatches(path, "/"));
+        sdd = path.equals(ROOT_PATH)? "0": Integer.toString(StringUtils.countMatches(path, "/"));
         break;
       case SASTokenProvider.GET_PROPERTIES_OPERATION:
       case SASTokenProvider.READ_OPERATION: