azure-sdk-for-net icon indicating copy to clipboard operation
azure-sdk-for-net copied to clipboard

[FEATURE REQ] SqlDatabaseResource.ExportAsync() / DatabaseExportDefinition to allow Managed Identity

Open johnburgato opened this issue 1 year ago • 3 comments
trafficstars

Library name

Azure.ResourceManager.Sql

Please describe the feature.

Currently, the DatabaseExportDefinition class constructor will only accept an Azure Storage access key and SQL Server username and password.

Two of the recommendations from Microsoft Defender for Cloud are: "Storage accounts should prevent shared key access" and "Azure SQL Database should have Azure Active Directory Only Authentication enabled". Both of these are considered "critical" recommendations.

We have a WebJob that triggers a database export each night, which I don't think is an un-common scenario. With the export functionality as it is, we could not possibly achieve the two recommendations of Microsoft Defender for Cloud. This seems especially important on the destination storage account, because it contains customer data in .bacpac files not protected in any other way.

johnburgato avatar Jun 27 '24 13:06 johnburgato

Thank you for your feedback. Tagging and routing to the team member best able to assist.

jsquire avatar Jun 27 '24 15:06 jsquire

Hi @johnburgato , .NET SDK is auto-generated from service swagger, if you need service to support the Export operation with ManagedIdentity, you could open an Azure support request to get further help.

HarveyLink avatar Jul 02 '24 06:07 HarveyLink

Hi @johnburgato. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text "/unresolve" to remove the "issue-addressed" label and continue the conversation.

github-actions[bot] avatar Jul 02 '24 06:07 github-actions[bot]

Hi @johnburgato, since you haven’t asked that we /unresolve the issue, we’ll close this out. If you believe further discussion is needed, please add a comment /unresolve to reopen the issue.

github-actions[bot] avatar Jul 09 '24 10:07 github-actions[bot]