Skip to content

Commit de43661

Browse files
committed
Updated auth service secret
1 parent 099a85e commit de43661

2 files changed

Lines changed: 1 addition & 8 deletions

File tree

lambdas/shared/src/common/api_clients/authentication.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@
1818
from common.clients import logger
1919
from common.models.errors import UnhandledResponseError
2020

21-
SERVICE_SECRETS_CACHE_TTL_SECONDS = 10 * 60
22-
2321

2422
class AppRestrictedAuth:
2523
def __init__(self, secret_manager_client: Any, environment: str, secret_name: str | None = None):
@@ -28,7 +26,6 @@ def __init__(self, secret_manager_client: Any, environment: str, secret_name: st
2826
self.cached_access_token: str | None = None
2927
self.cached_access_token_expiry_time: int | None = None
3028
self.cached_service_secrets: dict[str, Any] | None = None
31-
self.cached_service_secrets_expiry_time: int | None = None
3229

3330
self.secret_name = f"imms/outbound/{environment}/jwt-secrets" if secret_name is None else secret_name
3431
self.token_url = (
@@ -38,17 +35,14 @@ def __init__(self, secret_manager_client: Any, environment: str, secret_name: st
3835
)
3936

4037
def get_service_secrets(self) -> dict[str, Any]:
41-
now = int(time.time())
42-
43-
if self.cached_service_secrets and self.cached_service_secrets_expiry_time > now:
38+
if self.cached_service_secrets is not None:
4439
return self.cached_service_secrets
4540

4641
response = self.secret_manager_client.get_secret_value(SecretId=self.secret_name)
4742
secret_object = json.loads(response["SecretString"])
4843
secret_object["private_key"] = base64.b64decode(secret_object["private_key_b64"]).decode()
4944

5045
self.cached_service_secrets = secret_object
51-
self.cached_service_secrets_expiry_time = now + SERVICE_SECRETS_CACHE_TTL_SECONDS
5246
return secret_object
5347

5448
def create_jwt(self, now: int) -> str:

lambdas/shared/tests/test_common/api_clients/test_authentication.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,6 @@ def test_returned_cached_service_secrets(self):
115115
"private_key": self.private_key,
116116
}
117117
self.authenticator.cached_service_secrets = cached_secrets
118-
self.authenticator.cached_service_secrets_expiry_time = int(time.time()) + 99999
119118

120119
# When
121120
act_secrets = self.authenticator.get_service_secrets()

0 commit comments

Comments
 (0)