Skip to content

Commit 9ba4be8

Browse files
committed
updating log and removing dead code
1 parent 0f07dd8 commit 9ba4be8

File tree

1 file changed

+1
-19
lines changed

1 file changed

+1
-19
lines changed

prometrix/connect/aws_connect.py

Lines changed: 1 addition & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ def _build_auth(self) -> SigV4Auth:
101101
"""Builds fresh SigV4 auth with current credentials (handles rotation)."""
102102
try:
103103
if self._last_init_at is None or (datetime.utcnow() - self._last_init_at).total_seconds() >= AWS_REFRESH_CREDS_SEC:
104-
logging.debug("Fifteen minutes passed; re-initializing AWS credentials")
104+
logging.debug("%d seconds passed; re-initializing AWS credentials", AWS_REFRESH_CREDS_SEC)
105105
self.init_credentials()
106106
except Exception:
107107
logging.exception("Time-based credential refresh failed")
@@ -123,24 +123,6 @@ def signed_request(
123123
params=params,
124124
)
125125

126-
def _refresh_credentials(self) -> None:
127-
"""
128-
Boto should automatically refresh expired credentials but when assuming role it cant be done automatically
129-
"""
130-
try:
131-
if not self._has_static_keys and self._session is not None:
132-
# this is also needed for assume role if base credentials fails
133-
refreshed = self._session.get_credentials()
134-
if refreshed:
135-
self._credentials = refreshed
136-
except Exception:
137-
logging.exception("Failed to refresh session credentials")
138-
if self._role_to_assume:
139-
try:
140-
self._assume_role(self._role_to_assume)
141-
except Exception:
142-
logging.exception("Failed to refresh assume role")
143-
144126
def _request_with_refresh(self, *, method, url, data=None, params=None, headers=None, verify=False):
145127
resp = self.signed_request(
146128
method=method,

0 commit comments

Comments
 (0)