fix(global services): fixed global services region (#2203)

Co-authored-by: Pepe Fagoaga <pepe@verica.io>

prowler/providers/aws/services/account/account_service.py

@@ -1,12 +1,20 @@
+from prowler.providers.aws.aws_provider import generate_regional_clients
+
+
 ################## Account
-
-
 class Account:
     def __init__(self, audit_info):
         self.service = "account"
         self.session = audit_info.audit_session
         self.audited_account = audit_info.audited_account
-        self.region = audit_info.profile_region
+        self.regional_clients = generate_regional_clients(self.service, audit_info)
+        # If the region is not set in the audit profile,
+        # we pick the first region from the regional clients list
+        self.region = (
+            audit_info.profile_region
+            if audit_info.profile_region
+            else list(self.regional_clients.keys())[0]
+        )
 
     def __get_session__(self):
         return self.session

prowler/providers/aws/services/backup/backup_plans_exist/backup_plans_exist.py

@@ -10,7 +10,7 @@ class backup_plans_exist(Check):
         report.status_extended = "No Backup Plan Exist"
         report.resource_arn = ""
         report.resource_id = "No Backups"
-        report.region = backup_client.general_region
+        report.region = backup_client.region
         if backup_client.backup_plans:
             report.status = "PASS"
             report.status_extended = f"At least one backup plan exists: { backup_client.backup_plans[0].name}"

prowler/providers/aws/services/backup/backup_reportplans_exist/backup_reportplans_exist.py

@@ -10,7 +10,7 @@ class backup_reportplans_exist(Check):
         report.status_extended = "No Backup Report Plan Exist"
         report.resource_arn = ""
         report.resource_id = "No Backups"
-        report.region = backup_client.general_region
+        report.region = backup_client.region
         if backup_client.backup_report_plans:
             report.status = "PASS"
             report.status_extended = f"At least one backup report plan exists: { backup_client.backup_report_plans[0].name}"

prowler/providers/aws/services/backup/backup_service.py

@@ -16,7 +16,13 @@ class Backup:
         self.audited_account = audit_info.audited_account
         self.audit_resources = audit_info.audit_resources
         self.regional_clients = generate_regional_clients(self.service, audit_info)
-        self.general_region = audit_info.profile_region
+        # If the region is not set in the audit profile,
+        # we pick the first region from the regional clients list
+        self.region = (
+            audit_info.profile_region
+            if audit_info.profile_region
+            else list(self.regional_clients.keys())[0]
+        )
         self.backup_vaults = []
         self.__threading_call__(self.__list_backup_vaults__)
         self.backup_plans = []

prowler/providers/aws/services/backup/backup_vaults_exist/backup_vaults_exist.py

@@ -10,7 +10,7 @@ class backup_vaults_exist(Check):
         report.status_extended = "No Backup Vault Exist"
         report.resource_arn = ""
         report.resource_id = "No Backups"
-        report.region = backup_client.general_region
+        report.region = backup_client.region
         if backup_client.backup_vaults:
             report.status = "PASS"
             report.status_extended = f"At least one backup vault exists: { backup_client.backup_vaults[0].name}"

prowler/providers/aws/services/cloudtrail/cloudtrail_service.py

@@ -17,8 +17,14 @@ class Cloudtrail:
         self.audited_account = audit_info.audited_account
         self.audit_resources = audit_info.audit_resources
         self.audited_partition = audit_info.audited_partition
-        self.region = audit_info.profile_region
         self.regional_clients = generate_regional_clients(self.service, audit_info)
+        # If the region is not set in the audit profile,
+        # we pick the first region from the regional clients list
+        self.region = (
+            audit_info.profile_region
+            if audit_info.profile_region
+            else list(self.regional_clients.keys())[0]
+        )
         self.trails = []
         self.__threading_call__(self.__get_trails__)
         self.__get_trail_status__()

tests/providers/aws/services/backup/backup_plans_exist/backup_plans_exist_test.py

@@ -9,7 +9,7 @@ AWS_REGION = "eu-west-1"
 class Test_backup_plans_exist:
     def test_no_backup_plans(self):
         backup_client = mock.MagicMock
-        backup_client.general_region = AWS_REGION
+        backup_client.region = AWS_REGION
         backup_client.backup_plans = []
         with mock.patch(
             "prowler.providers.aws.services.backup.backup_service.Backup",
@@ -32,7 +32,7 @@ class Test_backup_plans_exist:
 
     def test_one_backup_plan(self):
         backup_client = mock.MagicMock
-        backup_client.general_region = AWS_REGION
+        backup_client.region = AWS_REGION
         backup_client.backup_plans = [
             BackupPlan(
                 arn="ARN",

tests/providers/aws/services/backup/backup_reportplans_exist/backup_reportplans_exist_test.py

@@ -9,7 +9,7 @@ AWS_REGION = "eu-west-1"
 class Test_backup_reportplans_exist:
     def test_no_backup_report_plans(self):
         backup_client = mock.MagicMock
-        backup_client.general_region = AWS_REGION
+        backup_client.region = AWS_REGION
         backup_client.backup_report_plans = []
         with mock.patch(
             "prowler.providers.aws.services.backup.backup_service.Backup",
@@ -32,7 +32,7 @@ class Test_backup_reportplans_exist:
 
     def test_one_backup_report_plan(self):
         backup_client = mock.MagicMock
-        backup_client.general_region = AWS_REGION
+        backup_client.region = AWS_REGION
         backup_client.backup_report_plans = [
             BackupReportPlan(
                 arn="ARN",

tests/providers/aws/services/backup/backup_vaults_exist/backup_vaults_exist_test.py

@@ -8,7 +8,7 @@ AWS_REGION = "eu-west-1"
 class Test_backup_vaults_exist:
     def test_no_backup_vaults(self):
         backup_client = mock.MagicMock
-        backup_client.general_region = AWS_REGION
+        backup_client.region = AWS_REGION
         backup_client.backup_vaults = []
         with mock.patch(
             "prowler.providers.aws.services.backup.backup_service.Backup",
@@ -31,7 +31,7 @@ class Test_backup_vaults_exist:
 
     def test_one_backup_vault(self):
         backup_client = mock.MagicMock
-        backup_client.general_region = AWS_REGION
+        backup_client.region = AWS_REGION
         backup_client.backup_vaults = [
             BackupVault(
                 arn="ARN",