Skip to content

Commit 20698ab

Browse files
Merge pull request #253170 from CESANU/patch-16
Update powershell-sample-vulnerability-assessment-azure-sql.md
2 parents acba844 + fd714d9 commit 20698ab

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

articles/defender-for-cloud/powershell-sample-vulnerability-assessment-azure-sql.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -521,12 +521,12 @@ if ($haveVaSetting) {
521521
# Set new SQL Vulnerability Assessment Setting
522522
LogMessage -LogMessage "Add express configuration Vulnerability Assessment feature setting for '$($ServerName)' server."
523523
$Response = SetSqlVulnerabilityAssessmentServerSetting -SubscriptionId $SubscriptionId -ResourceGroupName $ResourceGroupName -ServerName $ServerName
524-
525-
if ($Response.Content.Contains("Enabled")) {
524+
$successStatusCodes = @(200, 201, 202)
525+
if ($Response.StatusCode -in $successStatusCodes) {
526526
LogMessage -LogMessage "Congratulations, your server '$($ServerName)' server is set up with express configuration Vulnerability Assessment feature"
527527
}
528528
else {
529-
LogMessage -LogMessage "There was a problem to enable express configuration Vulnerability Assessment feature on the '$($ServerName)' server, please try again"
529+
LogMessage -LogMessage "There was a problem to enable express configuration Vulnerability Assessment feature on the '$($ServerName)' server. Error '$($Response.StatusCode)': '$($Response.Content)'"
530530
return
531531
}
532532

0 commit comments

Comments
 (0)