mirror of
https://github.com/prowler-cloud/prowler.git
synced 2026-07-04 19:21:51 +00:00
fix(typo): minor language updates (#7843)
This commit is contained in:
+1
-1
@@ -12,7 +12,7 @@ class sqlserver_auditing_enabled(Check):
|
||||
)
|
||||
report.subscription = subscription
|
||||
report.status = "PASS"
|
||||
report.status_extended = f"SQL Server {sql_server.name} from subscription {subscription} has a auditing policy configured."
|
||||
report.status_extended = f"SQL Server {sql_server.name} from subscription {subscription} has an auditing policy configured."
|
||||
for auditing_policy in sql_server.auditing_policies:
|
||||
if auditing_policy.state == "Disabled":
|
||||
report.status = "FAIL"
|
||||
|
||||
+3
-3
@@ -9,13 +9,13 @@ from prowler.providers.m365.services.entra.entra_service import (
|
||||
|
||||
|
||||
class entra_identity_protection_sign_in_risk_enabled(Check):
|
||||
"""Check if at least one Conditional Access policy is a Identity Protection sign-in risk policy.
|
||||
"""Check if at least one Conditional Access policy is an Identity Protection sign-in risk policy.
|
||||
|
||||
This check ensures that at least one Conditional Access policy is a Identity Protection sign-in risk policy.
|
||||
This check ensures that at least one Conditional Access policy is an Identity Protection sign-in risk policy.
|
||||
"""
|
||||
|
||||
def execute(self) -> list[CheckReportM365]:
|
||||
"""Execute the check to ensure that at least one Conditional Access policy is a Identity Protection sign-in risk policy.
|
||||
"""Execute the check to ensure that at least one Conditional Access policy is an Identity Protection sign-in risk policy.
|
||||
|
||||
Returns:
|
||||
list[CheckReportM365]: A list containing the results of the check.
|
||||
|
||||
+7
-7
@@ -9,13 +9,13 @@ from prowler.providers.m365.services.entra.entra_service import (
|
||||
|
||||
|
||||
class entra_identity_protection_user_risk_enabled(Check):
|
||||
"""Check if at least one Conditional Access policy is a Identity Protection user risk policy.
|
||||
"""Check if at least one Conditional Access policy is an Identity Protection user risk policy.
|
||||
|
||||
This check ensures that at least one Conditional Access policy is a Identity Protection user risk policy.
|
||||
This check ensures that at least one Conditional Access policy is an Identity Protection user risk policy.
|
||||
"""
|
||||
|
||||
def execute(self) -> list[CheckReportM365]:
|
||||
"""Execute the check to ensure that at least one Conditional Access policy is a Identity Protection user risk policy.
|
||||
"""Execute the check to ensure that at least one Conditional Access policy is an Identity Protection user risk policy.
|
||||
|
||||
Returns:
|
||||
list[CheckReportM365]: A list containing the results of the check.
|
||||
@@ -29,7 +29,7 @@ class entra_identity_protection_user_risk_enabled(Check):
|
||||
resource_id="conditionalAccessPolicies",
|
||||
)
|
||||
report.status = "FAIL"
|
||||
report.status_extended = "No Conditional Access Policy is an user risk based Identity Protection Policy."
|
||||
report.status_extended = "No Conditional Access Policy is a user risk based Identity Protection Policy."
|
||||
|
||||
for policy in entra_client.conditional_access_policies.values():
|
||||
if policy.state == ConditionalAccessPolicyState.DISABLED:
|
||||
@@ -62,13 +62,13 @@ class entra_identity_protection_user_risk_enabled(Check):
|
||||
)
|
||||
if RiskLevel.HIGH not in policy.conditions.user_risk_levels:
|
||||
report.status = "FAIL"
|
||||
report.status_extended = f"Conditional Access Policy '{policy.display_name}' is an user risk based Identity Protection Policy but does not protect against high risk potential account compromises."
|
||||
report.status_extended = f"Conditional Access Policy '{policy.display_name}' is a user risk based Identity Protection Policy but does not protect against high risk potential account compromises."
|
||||
elif policy.state == ConditionalAccessPolicyState.ENABLED_FOR_REPORTING:
|
||||
report.status = "FAIL"
|
||||
report.status_extended = f"Conditional Access Policy '{policy.display_name}' is an user risk based Identity Protection Policy and reports high risk potential account compromises, but does not protect against them."
|
||||
report.status_extended = f"Conditional Access Policy '{policy.display_name}' is a user risk based Identity Protection Policy and reports high risk potential account compromises, but does not protect against them."
|
||||
else:
|
||||
report.status = "PASS"
|
||||
report.status_extended = f"Conditional Access Policy '{policy.display_name}' is an user risk based Identity Protection Policy and does protect against high risk potential account compromises."
|
||||
report.status_extended = f"Conditional Access Policy '{policy.display_name}' is a user risk based Identity Protection Policy and does protect against high risk potential account compromises."
|
||||
break
|
||||
|
||||
findings.append(report)
|
||||
|
||||
+1
-1
@@ -12,7 +12,7 @@ class compute_instance_login_user(Check):
|
||||
)
|
||||
report.status = "PASS"
|
||||
report.status_extended = (
|
||||
f"VM Instance {instance.name} has a appropriate login user."
|
||||
f"VM Instance {instance.name} has an appropriate login user."
|
||||
)
|
||||
if instance.login_user:
|
||||
report.status = "FAIL"
|
||||
|
||||
+1
-1
@@ -122,7 +122,7 @@ class Test_sqlserver_auditing_enabled:
|
||||
assert result[0].status == "PASS"
|
||||
assert (
|
||||
result[0].status_extended
|
||||
== f"SQL Server {sql_server_name} from subscription {AZURE_SUBSCRIPTION_ID} has a auditing policy configured."
|
||||
== f"SQL Server {sql_server_name} from subscription {AZURE_SUBSCRIPTION_ID} has an auditing policy configured."
|
||||
)
|
||||
assert result[0].subscription == AZURE_SUBSCRIPTION_ID
|
||||
assert result[0].resource_name == sql_server_name
|
||||
|
||||
+5
-5
@@ -45,7 +45,7 @@ class Test_entra_identity_protection_user_risk_enabled:
|
||||
assert result[0].status == "FAIL"
|
||||
assert (
|
||||
result[0].status_extended
|
||||
== "No Conditional Access Policy is an user risk based Identity Protection Policy."
|
||||
== "No Conditional Access Policy is a user risk based Identity Protection Policy."
|
||||
)
|
||||
assert result[0].resource == {}
|
||||
assert result[0].resource_name == "Conditional Access Policies"
|
||||
@@ -119,7 +119,7 @@ class Test_entra_identity_protection_user_risk_enabled:
|
||||
assert result[0].status == "FAIL"
|
||||
assert (
|
||||
result[0].status_extended
|
||||
== "No Conditional Access Policy is an user risk based Identity Protection Policy."
|
||||
== "No Conditional Access Policy is a user risk based Identity Protection Policy."
|
||||
)
|
||||
assert result[0].resource == {}
|
||||
assert result[0].resource_name == "Conditional Access Policies"
|
||||
@@ -198,7 +198,7 @@ class Test_entra_identity_protection_user_risk_enabled:
|
||||
assert result[0].status == "FAIL"
|
||||
assert (
|
||||
result[0].status_extended
|
||||
== f"Conditional Access Policy '{display_name}' is an user risk based Identity Protection Policy but does not protect against high risk potential account compromises."
|
||||
== f"Conditional Access Policy '{display_name}' is a user risk based Identity Protection Policy but does not protect against high risk potential account compromises."
|
||||
)
|
||||
assert (
|
||||
result[0].resource
|
||||
@@ -280,7 +280,7 @@ class Test_entra_identity_protection_user_risk_enabled:
|
||||
assert result[0].status == "FAIL"
|
||||
assert (
|
||||
result[0].status_extended
|
||||
== f"Conditional Access Policy '{display_name}' is an user risk based Identity Protection Policy and reports high risk potential account compromises, but does not protect against them."
|
||||
== f"Conditional Access Policy '{display_name}' is a user risk based Identity Protection Policy and reports high risk potential account compromises, but does not protect against them."
|
||||
)
|
||||
assert (
|
||||
result[0].resource
|
||||
@@ -362,7 +362,7 @@ class Test_entra_identity_protection_user_risk_enabled:
|
||||
assert result[0].status == "PASS"
|
||||
assert (
|
||||
result[0].status_extended
|
||||
== f"Conditional Access Policy '{display_name}' is an user risk based Identity Protection Policy and does protect against high risk potential account compromises."
|
||||
== f"Conditional Access Policy '{display_name}' is a user risk based Identity Protection Policy and does protect against high risk potential account compromises."
|
||||
)
|
||||
assert (
|
||||
result[0].resource
|
||||
|
||||
+1
-1
@@ -67,7 +67,7 @@ class Test_compute_instance_login_user:
|
||||
|
||||
assert len(result) == 1
|
||||
assert result[0].status == "PASS"
|
||||
assert "has a appropriate login user" in result[0].status_extended
|
||||
assert "has an appropriate login user" in result[0].status_extended
|
||||
assert result[0].resource_name == instance_name
|
||||
assert result[0].resource_id == instance_id
|
||||
|
||||
|
||||
Reference in New Issue
Block a user