fix(Accounts): log access warning, do not raise in container .keys

NOTE: This will make sure that Ape API issues do not raise e.g.
`ape accounts list --all` and tracing features

Author: fubuloubu

ape_aws/accounts.py

@@ -2,6 +2,7 @@
 from typing import Any, Iterator, Optional
 
 from ape.api.accounts import AccountAPI, AccountContainerAPI, TransactionAPI
+from ape.logging import logger
 from ape.types import AddressType, MessageSignature, SignableMessage, TransactionSignature
 from eth_account._utils.legacy_transactions import serializable_unsigned_transaction_from_dict
 from eth_account.messages import _hash_eip191_message, encode_defunct
@@ -11,6 +12,7 @@
 from ape_aws.exceptions import ApeAwsException
 
 from .client import AwsClient
+from .exceptions import AwsAccessError
 from .kms.client import KmsKey
 from .utils import _convert_der_to_rsv
 
@@ -20,6 +22,16 @@ def __init__(self, *args, **kwargs):
         super(AwsClient, self).__init__()  # NOTE: Use config/envvar default
         super(AccountContainerAPI, self).__init__(*args, **kwargs)
 
+    @property
+    def keys(self) -> dict[str, KmsKey]:  # type: ignore[syntax]
+        try:
+            return super(AwsClient, self).keys
+
+        except AwsAccessError as e:
+            # NOTE: Do not raise here, instead just log warning (prevent issues w/ Ape API)
+            logger.warning(str(e))
+            return {}
+
     @property
     def aliases(self) -> Iterator[str]:
         yield from iter(self.keys)