Apply suggestions from code review

Co-authored-by: Viren Nadkarni <viren.nadkarni@localstack.cloud>
localstack · dfangl · Mar 13, 2023 · Nov 24, 2022 · Jan 2, 2023 · Jan 4, 2023
commit f10f9830142cf5f7cecfa30ba88a53c94258c9b7
diff --git a/localstack/aws/accounts.py b/localstack/aws/accounts.py
@@ -46,7 +46,7 @@ def set_aws_account_id(account_id: str) -> None:
     REQUEST_CTX_TLS.account_id = account_id
 
 
-def get_account_id_from_access_key_id(access_key_id: str) -> Optional[str]:
+def get_account_id_from_access_key_id(access_key_id: str) -> str:
     """Return the Account ID associated the Access Key ID."""
 
     # If AWS_ACCESS_KEY_ID has a 12-digit integer value, use it as the account ID

diff --git a/localstack/aws/handlers/internal_requests.py b/localstack/aws/handlers/internal_requests.py
@@ -20,7 +20,7 @@ class InternalRequestParamsEnricher(Handler):
     def __call__(self, chain: HandlerChain, context: RequestContext, response: Response):
         if header := context.request.headers.get(INTERNAL_REQUEST_PARAMS_HEADER):
             try:
-                dto = load_dto(header)
+                dto = MappingProxyType(load_dto(header))
             except Exception as e:
                 LOG.error("Error loading request parameters '%s', Error: %s", header, e)
                 return