modify get-azurestoragecontainer fix per review comments

Author: blueww

src/Storage/Commands.Storage/Blob/Cmdlet/GetAzureStorageContainer.cs

@@ -206,11 +206,15 @@ internal void PackCloudBlobContainerWithAcl(IEnumerable<Tuple<CloudBlobContainer
                 return;
             }
 
-            // Write warning when user SAS credential since get container ACL will fail
-            AzureStorageContext storageContext = this.GetCmdletStorageContext();
-            if (storageContext != null && storageContext.StorageAccount != null && storageContext.StorageAccount.Credentials != null && storageContext.StorageAccount.Credentials.IsSAS)
+            // Only write warning for SAS when use cmdlet alias Get-AzureStorageContainerAcl, since the cmdlets alias specified get container ACL
+            if (this.MyInvocation.Line.ToLower().Contains("get-azurestoragecontaineracl"))
             {
-                WriteWarning("Get Container ACL will fail with SAS token credentials.");
+                // Write warning when user SAS credential since get container ACL will fail
+                AzureStorageContext storageContext = this.GetCmdletStorageContext();
+                if (storageContext != null && storageContext.StorageAccount != null && storageContext.StorageAccount.Credentials != null && storageContext.StorageAccount.Credentials.IsSAS)
+                {
+                    WriteWarning("Get container permission will fail with SAS token credentials, it needs storage Account key credentials.");
+                }
             }
 
             IStorageBlobManagement localChannel = Channel;
@@ -238,13 +242,10 @@ internal async Task GetContainerPermission(long taskId, IStorageBlobManagement l
                 permissions = await localChannel.GetContainerPermissionsAsync(container, accessCondition,
                     requestOptions, OperationContext, CmdletCancellationToken).ConfigureAwait(false);
             }
-            catch (StorageException e)
+            catch (StorageException e) when (e.IsNotFoundException() || e.IsForbiddenException())
             {
-                if (!e.IsNotFoundException() && !e.IsFordiddenException())
-                {
-                    throw;
-                }
-                //404 Not found, or 403 Forbidden means we don't have permission to query the Permission of the specified container.
+                // 404 Not found, or 403 Forbidden means we don't have permission to query the Permission of the specified container.
+                // Just skip return container permission in this case.
             }
             WriteCloudContainerObject(taskId, localChannel, container, permissions, continuationToken);
         }

src/Storage/Commands.Storage/Common/StorageExceptionUtil.cs

@@ -37,7 +37,7 @@ public static bool IsNotFoundException(this StorageException e)
         /// </summary>
         /// <param name="e">StorageException from storage client</param>
         /// <returns>Whether the storageexception is caused by Fordidden</returns>
-        public static bool IsFordiddenException(this StorageException e)
+        public static bool IsForbiddenException(this StorageException e)
         {
             return e.RequestInformation != null && e.RequestInformation.HttpStatusCode == 403;
         }