From 0164574fddc87537f598d09c445aed20e0afad25 Mon Sep 17 00:00:00 2001
From: Sergio Garcia <38561120+sergargar@users.noreply.github.com>
Date: Fri, 3 Feb 2023 12:28:23 +0100
Subject: [PATCH] fix(KeyError): handle service key errors (#1831)

---
 .../aws/services/opensearch/opensearch_service.py      | 10 ++++++----
 .../s3_bucket_public_access/s3_bucket_public_access.py |  3 ++-
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/prowler/providers/aws/services/opensearch/opensearch_service.py b/prowler/providers/aws/services/opensearch/opensearch_service.py
index cab6a989..4e083f3c 100644
--- a/prowler/providers/aws/services/opensearch/opensearch_service.py
+++ b/prowler/providers/aws/services/opensearch/opensearch_service.py
@@ -94,10 +94,12 @@ class OpenSearchService:
                     DomainName=domain.name
                 )
                 domain.arn = describe_domain["DomainStatus"]["ARN"]
-                if "vpc" in describe_domain["DomainStatus"]["Endpoints"]:
-                    domain.endpoint_vpc = describe_domain["DomainStatus"]["Endpoints"][
-                        "vpc"
-                    ]
+                domain.endpoint_vpc = None
+                if "Endpoints" in describe_domain["DomainStatus"]:
+                    if "vpc" in describe_domain["DomainStatus"]["Endpoints"]:
+                        domain.endpoint_vpc = describe_domain["DomainStatus"][
+                            "Endpoints"
+                        ]["vpc"]
                 domain.vpc_id = describe_domain["DomainStatus"]["VPCOptions"]["VPCId"]
                 domain.cognito_options = describe_domain["DomainStatus"][
                     "CognitoOptions"
diff --git a/prowler/providers/aws/services/s3/s3_bucket_public_access/s3_bucket_public_access.py b/prowler/providers/aws/services/s3/s3_bucket_public_access/s3_bucket_public_access.py
index 4fbe1e48..3888426b 100644
--- a/prowler/providers/aws/services/s3/s3_bucket_public_access/s3_bucket_public_access.py
+++ b/prowler/providers/aws/services/s3/s3_bucket_public_access/s3_bucket_public_access.py
@@ -45,7 +45,8 @@ class s3_bucket_public_access(Check):
                     if bucket.policy:
                         for statement in bucket.policy["Statement"]:
                             if (
-                                "*" == statement["Principal"]
+                                "Principal" in statement
+                                and "*" == statement["Principal"]
                                 and statement["Effect"] == "Allow"
                             ):
                                 report.status = "FAIL"