Merge pull request #7820 from yaakoviyun/preview

Removed storage depedency when inferring storageBlobContainerName

Author: Maddie Clayton

src/ResourceManager/Sql/Commands.Sql/ChangeLog.md

@@ -18,6 +18,7 @@
         - Additional information about change #1
 -->
 ## Current Release
+* Fixed issue with using Get-AzureRmSqlDatabaseVulnerabilityAssessment with DotNet core
 
 ## Version 4.12.0
 * Added new cmdlets for CRUD operations on Azure Sql Database Managed Instance and Azure Sql Managed Database

src/ResourceManager/Sql/Commands.Sql/VulnerabilityAssessment/Services/BaseSqlVulnerabilityAssessmentAdapter.cs

@@ -84,16 +84,14 @@ public DatabaseVulnerabilityAssessmentSettingsModel ConvertSettingsToModel(strin
             string storageBlobContainerName = string.Empty;
             if (!string.IsNullOrEmpty(settings.StorageContainerPath))
             {
+                // StorageContainerPath is in the format of : "https://va1storage.blob.core.windows.net/vulnerability-assessment"
                 string storageAccountNamePart = settings.StorageContainerPath.Split(new string[] { "https://" }, StringSplitOptions.RemoveEmptyEntries)[0];
                 if (!string.IsNullOrEmpty(storageAccountNamePart))
                 {
                     storageAccountName = storageAccountNamePart.Split(new string[] { "." }, StringSplitOptions.RemoveEmptyEntries)[0];
                 }
 
-#if!NETSTANDARD
-                CloudBlob cloudBlob = new CloudBlob(new Uri(settings.StorageContainerPath));
-                storageBlobContainerName = cloudBlob.Container.Name;
-#endif
+                storageBlobContainerName = settings.StorageContainerPath.Split(new string[] { "/" }, StringSplitOptions.RemoveEmptyEntries).LastOrDefault();
             }
 
             var databaseVulnerabilityAssessmentSettingsModel = new DatabaseVulnerabilityAssessmentSettingsModel