{Sql} Fix Azure#20703: Fix 'Set-AzSqlServerAudit' missing WorkSpaceResourceId para… (Azure#20760)

RakeshMohanMSFT · web-flow · commit 12e4910a5b4f · 2023-01-28T16:07:02.000+08:00
* {Sql} Fix Azure#20703: Fix 'Set-AzSqlServerAudit' missing WorkSpaceId parameter Fixes Azure#20703 * {Sql} Fix Azure#20703: Fix 'Set-AzSqlServerAudit' missing WorkspaceResourceId parameter
diff --git a/src/Sql/Sql/Auditing/Cmdlet/SetSqlServerAuditCmdlet.cs b/src/Sql/Sql/Auditing/Cmdlet/SetSqlServerAuditCmdlet.cs
@@ -124,8 +124,7 @@ protected override ServerAuditModelType ApplyUserInputToModel(ServerAuditModelTy
 
             if (WorkspaceResourceId != null)
             {
-                model.LogAnalyticsTargetState = LogAnalyticsTargetState == SecurityConstants.Enabled ?
-                    AuditStateType.Enabled : AuditStateType.Disabled;
+                model.WorkspaceResourceId = WorkspaceResourceId;
             }
 
             if (UseIdentity != null)
diff --git a/src/Sql/Sql/ChangeLog.md b/src/Sql/Sql/ChangeLog.md
@@ -20,6 +20,7 @@
 ## Upcoming Release
 * Added an optional parameter `HAReplicaCount` to `Restore-AzSqlDatabase`
 * Enabled support for UserAssignedManagedIdentity in Auditing
+* Fixed WorkspaceResourceId parameter value in `Set-AzSqlServerAudit`
 
 ## Version 4.2.0
 * Added a parameter named `UseIdentity` for `Set-AzSqlServerAudit`, `Set-AzSqlDatabaseAudit`, `Set-AzSqlServerMSSupportAudit`

Original file line number	Diff line number	Diff line change
`@@ -124,8 +124,7 @@ protected override ServerAuditModelType ApplyUserInputToModel(ServerAuditModelTy`
`124`	`124`
`125`	`125`	`if (WorkspaceResourceId != null)`
`126`	`126`	`{`
`127`		`- model.LogAnalyticsTargetState = LogAnalyticsTargetState == SecurityConstants.Enabled ?`
`128`		`- AuditStateType.Enabled : AuditStateType.Disabled;`
	`127`	`+ model.WorkspaceResourceId = WorkspaceResourceId;`
`129`	`128`	`}`
`130`	`129`
`131`	`130`	`if (UseIdentity != null)`