Merge pull request Azure#10402 from bashahee/auditing

Fix a small bug.

Author: wyunchi-ms

src/Sql/Sql/Auditing/Services/SqlAuditAdapter.cs

@@ -134,8 +134,13 @@ private void ModelizeStorageInfo(ServerAuditModel model,
 
             if (isAuditEnabled)
             {
+                if (storageAccountSubscriptionId == null || Guid.Empty.Equals(storageAccountSubscriptionId))
+                {
+                    storageAccountSubscriptionId = Subscription.GetId();
+                }
+
                 model.StorageAccountResourceId = AzureCommunicator.RetrieveStorageAccountIdAsync(
-                    storageAccountSubscriptionId ?? Subscription.GetId(),
+                    (Guid)storageAccountSubscriptionId,
                     GetStorageAccountName(storageEndpoint)).GetAwaiter().GetResult();
                 ModelizeRetentionInfo(model, retentionDays);
             }

src/Sql/Sql/ChangeLog.md

@@ -30,6 +30,7 @@
 * Remove aliases for deprecated Vulnerability Assessment Settings cmdlets
 * Deprecate Advanced Threat Detection Settings cmdlets 
 * Adding cmdlets to Disable and enable sensitivity recommendations on columns in a database.
+* Fix a small bug when reading auditing settings of a server or a database.
 
 ## Version 1.15.0
 * Add support for setting Active Directory Administrator on Managed Instance