using shared value

Author: Andrei Neagu

services/storage/src/simcore_service_storage/constants.py

@@ -41,3 +41,6 @@
 
 
 EXPAND_DIR_MAX_ITEM_COUNT: Final[int] = 1000
+
+
+EXPORTS_S3_PREFIX: Final[str] = "exports"

services/storage/src/simcore_service_storage/modules/db/access_layer.py

@@ -53,6 +53,7 @@
 from simcore_postgres_database.utils_sql import assemble_array_groups
 from sqlalchemy.ext.asyncio import AsyncConnection
 
+from ...constants import EXPORTS_S3_PREFIX
 from ...exceptions.errors import InvalidFileIdentifierError
 from ...models import AccessRights
 from ._base import BaseRepository
@@ -335,7 +336,7 @@ async def get_file_access_rights(  # pylint:disable=too-many-return-statements #
                     # ownership still not defined, so we assume it is user_id
                     return AccessRights.all()
 
-                if parent == "exports":
+                if parent == EXPORTS_S3_PREFIX:
                     # ownership still not defined, so we assume it is user_id
                     return AccessRights.all()
 

services/storage/src/simcore_service_storage/utils/simcore_s3_dsm_utils.py

@@ -20,6 +20,7 @@
 from servicelib.utils import ensure_ends_with
 from sqlalchemy.ext.asyncio import AsyncEngine
 
+from ..constants import EXPORTS_S3_PREFIX
 from ..exceptions.errors import FileMetaDataNotFoundError, ProjectAccessRightError
 from ..models import FileMetaData, FileMetaDataAtDB, GenericCursor, PathMetaData
 from ..modules.db.access_layer import AccessLayerRepository
@@ -138,7 +139,7 @@ def compute_file_id_prefix(file_id: str, levels: int):
 
 def get_random_export_name(user_id: UserID) -> StorageFileID:
     return TypeAdapter(StorageFileID).validate_python(
-        f"exports/{user_id}/{uuid4()}.zip"
+        f"{EXPORTS_S3_PREFIX}/{user_id}/{uuid4()}.zip"
     )