Skip to content

Commit 9190bb1

Browse files
authored
Merge pull request #79 from SkillsFundingAgency/DASD-284
DASD-2844 SQL auditing
2 parents c163b3d + 514d53b commit 9190bb1

1 file changed

Lines changed: 13 additions & 2 deletions

File tree

Infrastructure/Resources/New-SQLServer.ps1

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,12 @@ if ($SQLServer) {
200200
RetentionInDays = 90
201201
State = "Enabled"
202202
}
203-
Set-AzureRmSqlServerAuditing @AuditingPolicyParameters
203+
$currentAuditingSetting = Get-AzureRmSqlServerAuditing -ResourceGroupName $ResourceGroupName -ServerName $ServerName
204+
if ($currentAuditingSetting.AuditState -ne $AuditingPolicyParameters.State `
205+
-or $currentAuditingSetting.RetentionInDays -ne $AuditingPolicyParameters.RetentionInDays `
206+
-or $currentAuditingSetting.StorageAccountName -ne $AuditingPolicyParameters.StorageAccountName) {
207+
Set-AzureRmSqlServerAuditing @AuditingPolicyParameters
208+
}
204209
}
205210
else {
206211
$AuditingPolicyParameters = @{
@@ -211,7 +216,13 @@ if ($SQLServer) {
211216
EventType = "All"
212217
RetentionInDays = 90
213218
}
214-
Set-AzureRmSqlServerAuditingPolicy @AuditingPolicyParameters
219+
$currentAuditingSetting = Get-AzureRmSqlServerAuditingPolicy -ResourceGroupName $ResourceGroupName -ServerName $ServerName
220+
if ($currentAuditingSetting.AuditState -ne $AuditingPolicyParameters.State `
221+
-or $currentAuditingSetting.AuditType -ne $AuditingPolicyParameters.AuditType `
222+
-or $currentAuditingSetting.RetentionInDays -ne $AuditingPolicyParameters.RetentionInDays `
223+
-or $currentAuditingSetting.StorageAccountName -ne $AuditingPolicyParameters.StorageAccountName) {
224+
Set-AzureRmSqlServerAuditingPolicy @AuditingPolicyParameters
225+
}
215226
}
216227

217228
Write-Log -LogLevel Information -Message "Configuring threat detection policy"

0 commit comments

Comments
 (0)