Skip to content

Commit 3bc5ebd

Browse files
authored
Merge pull request #188 from ecmwf-projects/fix-cache-race-condition
Fix cache race condition
2 parents 52a0478 + 8b4b38a commit 3bc5ebd

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cads_processing_api_service/auth.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,6 @@ def get_auth_header(
7878
maxsize=config.ensure_settings().cache_users_maxsize,
7979
ttl=config.ensure_settings().cache_users_ttl,
8080
),
81-
info=True,
8281
)
8382
def authenticate_user(
8483
auth_header: tuple[str, str], portal_header: str | None = None
@@ -156,7 +155,6 @@ def verify_permission(user_uid: str, job: cads_broker.SystemRequest) -> None:
156155
maxsize=config.ensure_settings().cache_users_maxsize,
157156
ttl=config.ensure_settings().cache_users_ttl,
158157
),
159-
info=True,
160158
)
161159
def get_accepted_licences(auth_header: tuple[str, str]) -> set[tuple[str, int]]:
162160
"""Get licences accepted by a user stored in the Extended Profiles database.

cads_processing_api_service/utils.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
import base64
1818
import datetime
1919
import enum
20+
import threading
2021
from typing import Any, Callable, Mapping
2122

2223
import cachetools
@@ -54,6 +55,7 @@ class JobSortCriterion(str, enum.Enum):
5455
maxsize=config.ensure_settings().cache_resources_maxsize,
5556
ttl=config.ensure_settings().cache_resources_ttl,
5657
),
58+
lock=threading.Lock(),
5759
key=lambda resource_id,
5860
table,
5961
session,

0 commit comments

Comments
 (0)