-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
⭐️ Azure: Microsoft Microsoft Defender for Cloud resource updates #4554
Conversation
HRouhani
commented
Aug 14, 2024
•
edited
Loading
edited
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @HRouhani!
I went through the Microsoft documentation on each of these components and changed the product names and capitalization to match Microsoft's.
Some of Microsoft's choices are weird and don't seem consistent But 🤷 if you want to match Microsoft, you can accept these suggestions. Up to you!
providers/azure/resources/azure.lr
Outdated
defenderForAppServices() dict | ||
// Whether Microsoft Defender for SQL Servers on Machines is enabled | ||
defenderForSqlServersOnMachines() dict | ||
// Whether Microsoft Defender for Azure SQL Databases is enabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Whether Microsoft Defender for Azure SQL Databases is enabled | |
// Whether Microsoft Defender for Azure SQL is enabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The focus is on " Azure SQL Databases" here
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Signed-off-by: Hossein Rouhani <h_rouhani@hotmail.com>
Co-authored-by: Letha <letha@mondoo.com>
Co-authored-by: Letha <letha@mondoo.com>
Co-authored-by: Letha <letha@mondoo.com>
Co-authored-by: Letha <letha@mondoo.com>
Co-authored-by: Letha <letha@mondoo.com>
Co-authored-by: Letha <letha@mondoo.com>
Co-authored-by: Letha <letha@mondoo.com>
7095054
to
7d39027
Compare
@@ -1697,9 +1697,25 @@ private azure.subscription.cloudDefenderService @defaults("defenderForServers de | |||
subscriptionId string | |||
// Whether the monitoring agent is automatically provisioned on new VMs | |||
monitoringAgentAutoProvision() bool | |||
// List of Defender for Server components and whether they are enabled | |||
// List of Defender for Servers components and whether they are enabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we cannot use boolean indicators for the text, when the output is non dict with multiple values.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @HRouhani