fix: add missing Check_Report_Azure parameters (#6583)

This commit is contained in:
Rubén De la Torre Vico
2025-01-17 14:16:43 +01:00
committed by Pepe Fagoaga
parent 18a4881a51
commit b1de41619b
3 changed files with 10 additions and 3 deletions
@@ -32,7 +32,10 @@ class entra_conditional_access_policy_require_mfa_for_management_api(Check):
)
break
else:
report = Check_Report_Azure(self.metadata())
report = Check_Report_Azure(
metadata=self.metadata(),
resource_metadata=conditional_access_policies.values(),
)
report.subscription = f"Tenant: {tenant_name}"
report.resource_name = "Conditional Access Policy"
report.resource_id = "Conditional Access Policy"
@@ -14,7 +14,9 @@ class entra_non_privileged_user_has_mfa(Check):
if not is_privileged_user(
user, entra_client.directory_roles[tenant_domain]
):
report = Check_Report_Azure(self.metadata(), resource_metadata=user)
report = Check_Report_Azure(
metadata=self.metadata(), resource_metadata=user
)
report.subscription = f"Tenant: {tenant_domain}"
report.status = "FAIL"
report.status_extended = (
@@ -10,7 +10,9 @@ class monitor_diagnostic_settings_exists(Check):
subscription_name,
diagnostic_settings,
) in monitor_client.diagnostics_settings.items():
report = Check_Report_Azure(self.metadata(), diagnostic_settings)
report = Check_Report_Azure(
metadata=self.metadata(), resource_metadata=diagnostic_settings
)
report.subscription = subscription_name
report.resource_name = "Diagnostic Settings"
report.resource_id = "diagnostic_settings"