data-syncmaster 0.2.1__tar.gz → 0.2.3__tar.gz

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: data-syncmaster
-Version: 0.2.1
+Version: 0.2.3
 Summary: Syncmaster REST API + Worker
 License: Apache-2.0
 Keywords: Syncmaster,REST,API,Worker,Spark,Transfer,ETL

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "data-syncmaster"
-version = "0.2.1"
+version = "0.2.3"
 license = "Apache-2.0"
 description = "Syncmaster REST API + Worker"
 authors = ["DataOps.ETL <onetools@mts.ru>"]

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/__init__.py

@@ -1,6 +1,6 @@
 # SPDX-FileCopyrightText: 2023-2024 MTS PJSC
 # SPDX-License-Identifier: Apache-2.0
 
-_raw_version = "0.2.1"
+_raw_version = "0.2.3"
 # version always contain only release number like 0.0.1
 __version__ = ".".join(_raw_version.split(".")[:3])  # noqa: WPS410

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/db/migrations/versions/2023-11-23_0002_create_group_table.py

@@ -22,7 +22,7 @@ def upgrade():
     op.create_table(
         "group",
         sa.Column("id", sa.BigInteger(), nullable=False),
-        sa.Column("name", sa.String(length=256), nullable=False),
+        sa.Column("name", sa.String(length=128), nullable=False),
         sa.Column("description", sa.String(length=512), nullable=False),
         sa.Column("owner_id", sa.BigInteger(), nullable=False),
         sa.Column("created_at", sa.DateTime(timezone=True), server_default=sa.text("now()"), nullable=False),

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/db/migrations/versions/2023-11-23_0004_create_connection_table.py

@@ -30,7 +30,7 @@ def upgrade():
         "connection",
         sa.Column("id", sa.BigInteger(), nullable=False),
         sa.Column("group_id", sa.BigInteger(), nullable=False),
-        sa.Column("type", sa.String(length=23), nullable=False),
+        sa.Column("type", sa.String(length=32), nullable=False),
         sa.Column("name", sa.String(length=128), nullable=False),
         sa.Column("description", sa.String(length=512), nullable=False),
         sa.Column("data", sa.JSON(), nullable=False),

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/db/models/connection.py

@@ -32,7 +32,7 @@ class ConnectionType(StrEnum):
 
 
 class Connection(Base, ResourceMixin, TimestampMixin):
-    type: Mapped[ConnectionType] = mapped_column(ChoiceType(ConnectionType, impl=String(23)), nullable=False)
+    type: Mapped[ConnectionType] = mapped_column(ChoiceType(ConnectionType, impl=String(32)), nullable=False)
     data: Mapped[dict[str, Any]] = mapped_column(JSON, nullable=False, default={})
 
     group: Mapped[Group] = relationship("Group")

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/db/models/group.py

@@ -71,7 +71,7 @@ class GroupMemberRole(enum.StrEnum):
 
 class Group(Base, TimestampMixin):
     id: Mapped[int] = mapped_column(BigInteger, primary_key=True)
-    name: Mapped[str] = mapped_column(String(256), nullable=False, unique=True)
+    name: Mapped[str] = mapped_column(String(128), nullable=False, unique=True)
     description: Mapped[str] = mapped_column(String(512), nullable=False, default="")
     owner_id: Mapped[int] = mapped_column(ForeignKey("user.id", ondelete="CASCADE"), nullable=False, index=True)
 

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/db/repositories/group.py

@@ -15,7 +15,6 @@ from syncmaster.exceptions import EntityNotFoundError, SyncmasterError
 from syncmaster.exceptions.group import (
     AlreadyIsGroupMemberError,
     AlreadyIsNotGroupMemberError,
-    GroupAdminNotFoundError,
     GroupAlreadyExistsError,
     GroupNotFoundError,
 )
@@ -364,7 +363,7 @@ class GroupRepository(Repository[Group]):
         constraint = err.__cause__.__cause__.constraint_name
 
         if constraint == "fk__group__owner_id__user":
-            raise GroupAdminNotFoundError from err
+            raise UserNotFoundError from err
 
         if constraint == "uq__group__name":
             raise GroupAlreadyExistsError from err

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/db/repositories/queue.py

@@ -73,10 +73,10 @@ class QueueRepository(RepositoryWithOwner[Queue]):
         queue_data: UpdateQueueSchema,
     ) -> Queue:
         try:
-            queue = await self.read_by_id(queue_id=queue_id)
             return await self._update(
                 Queue.id == queue_id,
-                description=queue_data.description or queue.description,
+                name=queue_data.name,
+                description=queue_data.description,
             )
         except IntegrityError as e:
             self._raise_error(e)

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/exceptions/group.py

@@ -7,10 +7,6 @@ class GroupNameAlreadyExistsError(SyncmasterError):
     pass
 
 
-class GroupAdminNotFoundError(SyncmasterError):
-    pass
-
-
 class GroupAlreadyExistsError(SyncmasterError):
     pass
 

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/__init__.py

@@ -55,7 +55,6 @@ from syncmaster.schemas.v1.users import (
     FullUserSchema,
     ReadGroupMember,
     ReadUserSchema,
-    UpdateUserSchema,
     UserPageSchema,
     UserPageSchemaAsGroupMember,
 )
@@ -100,7 +99,6 @@ __all__ = [
     "JSONLine",
     "Parquet",
     "ReadUserSchema",
-    "UpdateUserSchema",
     "FullUserSchema",
     "ReadGroupMember",
     "UserPageSchema",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/clickhouse.py

@@ -29,7 +29,7 @@ class ReadClickhouseConnectionDataSchema(BaseModel):
 
 
 class CreateClickhouseConnectionSchema(CreateConnectionBaseSchema):
-    type: CLICKHOUSE_TYPE = Field(..., description="Connection type")
+    type: CLICKHOUSE_TYPE = Field(description="Connection type")
     data: CreateClickhouseConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/connection.py

@@ -125,7 +125,7 @@ UpdateConnectionSchema = Annotated[
 
 class ConnectionCopySchema(BaseModel):
     new_group_id: int
-    new_name: NameConstr | None = None  # noqa: F722
+    new_name: NameConstr | None = None
     remove_source: bool = False
 
 

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/connection_base.py

@@ -10,9 +10,9 @@ ReadConnectionAuthDataSchema = ReadBasicAuthSchema | ReadS3AuthSchema | ReadSamb
 
 
 class CreateConnectionBaseSchema(BaseModel):
-    group_id: int = Field(..., description="Connection owner group id")
-    name: NameConstr = Field(..., description="Connection name")  # noqa: F722
-    description: str = Field(..., description="Additional description")
+    group_id: int = Field(description="Connection owner group id")
+    name: NameConstr = Field(description="Connection name")
+    description: str = Field(description="Additional description")
 
     model_config = ConfigDict(populate_by_name=True)
 

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/ftp.py

@@ -26,7 +26,7 @@ class ReadFTPConnectionDataSchema(BaseModel):
 
 
 class CreateFTPConnectionSchema(CreateConnectionBaseSchema):
-    type: FTP_TYPE = Field(..., description="Connection type")
+    type: FTP_TYPE = Field(description="Connection type")
     data: CreateFTPConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/ftps.py

@@ -26,7 +26,7 @@ class ReadFTPSConnectionDataSchema(BaseModel):
 
 
 class CreateFTPSConnectionSchema(CreateConnectionBaseSchema):
-    type: FTPS_TYPE = Field(..., description="Connection type")
+    type: FTPS_TYPE = Field(description="Connection type")
     data: CreateFTPSConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/hdfs.py

@@ -24,7 +24,7 @@ class ReadHDFSConnectionDataSchema(BaseModel):
 
 
 class CreateHDFSConnectionSchema(CreateConnectionBaseSchema):
-    type: HDFS_TYPE = Field(..., description="Connection type")
+    type: HDFS_TYPE = Field(description="Connection type")
     data: CreateHDFSConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/hive.py

@@ -24,7 +24,7 @@ class ReadHiveConnectionDataSchema(BaseModel):
 
 
 class CreateHiveConnectionSchema(CreateConnectionBaseSchema):
-    type: HIVE_TYPE = Field(..., description="Connection type")
+    type: HIVE_TYPE = Field(description="Connection type")
     data: CreateHiveConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/mssql.py

@@ -29,7 +29,7 @@ class ReadMSSQLConnectionDataSchema(BaseModel):
 
 
 class CreateMSSQLConnectionSchema(CreateConnectionBaseSchema):
-    type: MSSQL_TYPE = Field(..., description="Connection type")
+    type: MSSQL_TYPE = Field(description="Connection type")
     data: CreateMSSQLConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/mysql.py

@@ -29,7 +29,7 @@ class ReadMySQLConnectionDataSchema(BaseModel):
 
 
 class CreateMySQLConnectionSchema(CreateConnectionBaseSchema):
-    type: MYSQL_TYPE = Field(..., description="Connection type")
+    type: MYSQL_TYPE = Field(description="Connection type")
     data: CreateMySQLConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/oracle.py

@@ -39,7 +39,7 @@ class ReadOracleConnectionDataSchema(BaseModel):
 
 
 class CreateOracleConnectionSchema(CreateConnectionBaseSchema):
-    type: ORACLE_TYPE = Field(..., description="Connection type")
+    type: ORACLE_TYPE = Field(description="Connection type")
     data: CreateOracleConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/postgres.py

@@ -30,7 +30,7 @@ class ReadPostgresConnectionDataSchema(BaseModel):
 
 
 class CreatePostgresConnectionSchema(CreateConnectionBaseSchema):
-    type: POSTGRES_TYPE = Field(..., description="Connection type")
+    type: POSTGRES_TYPE = Field(description="Connection type")
     data: CreatePostgresConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/s3.py

@@ -46,7 +46,7 @@ class ReadS3ConnectionDataSchema(BaseModel):
 
 
 class CreateS3ConnectionSchema(CreateConnectionBaseSchema):
-    type: S3_TYPE = Field(..., description="Connection type")
+    type: S3_TYPE = Field(description="Connection type")
     data: CreateS3ConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/samba.py

@@ -33,7 +33,7 @@ class ReadSambaConnectionDataSchema(BaseModel):
 
 
 class CreateSambaConnectionSchema(CreateConnectionBaseSchema):
-    type: SAMBA_TYPE = Field(..., description="Connection type")
+    type: SAMBA_TYPE = Field(description="Connection type")
     data: CreateSambaConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/sftp.py

@@ -26,7 +26,7 @@ class ReadSFTPConnectionDataSchema(BaseModel):
 
 
 class CreateSFTPConnectionSchema(CreateConnectionBaseSchema):
-    type: SFTP_TYPE = Field(..., description="Connection type")
+    type: SFTP_TYPE = Field(description="Connection type")
     data: CreateSFTPConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/connections/webdav.py

@@ -30,7 +30,7 @@ class ReadWebDAVConnectionDataSchema(BaseModel):
 
 
 class CreateWebDAVConnectionSchema(CreateConnectionBaseSchema):
-    type: WEBDAV_TYPE = Field(..., description="Connection type")
+    type: WEBDAV_TYPE = Field(description="Connection type")
     data: CreateWebDAVConnectionDataSchema = Field(
         ...,
         alias="connection_data",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/groups.py

@@ -8,13 +8,13 @@ from syncmaster.schemas.v1.types import NameConstr
 
 
 class UpdateGroupSchema(BaseModel):
-    name: NameConstr  # noqa: F722
+    name: NameConstr
     description: str
     owner_id: int
 
 
 class CreateGroupSchema(BaseModel):
-    name: NameConstr  # noqa: F722
+    name: NameConstr
     description: str
 
 
@@ -40,7 +40,7 @@ class AddUserSchema(BaseModel):
 
 class ReadGroupSchema(BaseModel):
     id: int
-    name: NameConstr  # noqa: F722
+    name: str
     description: str
     owner_id: int
 

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/queue.py

@@ -1,22 +1,33 @@
 # SPDX-FileCopyrightText: 2023-2024 MTS PJSC
 # SPDX-License-Identifier: Apache-2.0
-from pydantic import BaseModel, ConfigDict, Field, computed_field, constr
+import re
+from typing import Annotated
+
+from pydantic import BaseModel, ConfigDict, Field, StringConstraints, computed_field
 
 from syncmaster.schemas.v1.page import PageSchema
 
+ALLOWED_PATTERN = re.compile(r"^[ -~]+$")
+RESTRICTED_PATTERN = re.compile(r"[^a-zA-Z0-9]+")
+
+QueueName = Annotated[
+    str,
+    StringConstraints(min_length=3, max_length=128, pattern=ALLOWED_PATTERN),  # noqa: WPS432
+]
+
 
 class CreateQueueSchema(BaseModel):
-    name: constr(max_length=128, pattern=r"^[-_a-zA-Z0-9]+$") = Field(  # noqa: F722, WPS432
-        ...,
+    name: QueueName = Field(
         description="Queue name that allows letters, numbers, dashes, and underscores",
     )
-    group_id: int = Field(..., description="Queue owner group id")
+    group_id: int = Field(description="Queue owner group id")
     description: str = Field(default="", description="Additional description")
 
     @computed_field
     @property
     def slug(self) -> str:
-        return f"{self.group_id}-{self.name}"
+        short_name = RESTRICTED_PATTERN.sub("_", self.name.lower()).strip("_")
+        return f"{self.group_id}-{short_name}"
 
 
 class ReadQueueSchema(BaseModel):
@@ -34,4 +45,5 @@ class QueuePageSchema(PageSchema):
 
 
 class UpdateQueueSchema(BaseModel):
-    description: str | None = None
+    name: QueueName
+    description: str = ""

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/transfers/__init__.py

@@ -142,9 +142,9 @@ TransformationSchema = DataframeRowsFilter | DataframeColumnsFilter | FileMetada
 class CopyTransferSchema(BaseModel):
     new_group_id: int
     new_queue_id: int
-    new_source_connection_name: NameConstr | None = None  # noqa: F722
-    new_target_connection_name: NameConstr | None = None  # noqa: F722
-    new_name: NameConstr | None = None  # noqa: F722
+    new_source_connection_name: NameConstr | None = None
+    new_target_connection_name: NameConstr | None = None
+    new_name: NameConstr | None = None
     remove_source: bool = False
 
 
@@ -176,14 +176,14 @@ class ReadTransferSchema(BaseModel):
 
 
 class CreateTransferSchema(BaseModel):
-    group_id: int = Field(..., description="Transfer owner group id")
-    source_connection_id: int = Field(..., description="id of the connection that will be the data source")
-    target_connection_id: int = Field(..., description="id of the connection that will be the data receiver")
-    name: NameConstr = Field(..., description="Transfer name")  # noqa: F722
-    description: str = Field(..., description="Additional description")
-    is_scheduled: bool = Field(..., description="Is the transfer on schedule")
-    queue_id: int = Field(..., description="id of the queue in which the transfer will be performed")
-    schedule: str | None = Field(None, description="Execution schedule in cron format")
+    group_id: int = Field(description="Transfer owner group id")
+    source_connection_id: int = Field(description="id of the connection that will be the data source")
+    target_connection_id: int = Field(description="id of the connection that will be the data receiver")
+    name: NameConstr = Field(description="Transfer name")
+    description: str = Field(description="Additional description")
+    is_scheduled: bool = Field(description="Is the transfer on schedule")
+    queue_id: int = Field(description="id of the queue in which the transfer will be performed")
+    schedule: str | None = Field(default=None, description="Execution schedule in cron format")
     source_params: CreateTransferSchemaSource = Field(
         ...,
         discriminator="type",

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/transfers/file/base.py

@@ -23,7 +23,7 @@ from syncmaster.schemas.v1.transfers.file_format import (
 # classes are identical but may change in the future
 class ReadFileTransferSource(BaseModel):
     directory_path: str
-    file_format: CSV | JSONLine | JSON | Excel | XML | ORC | Parquet = Field(..., discriminator="type")
+    file_format: CSV | JSONLine | JSON | Excel | XML | ORC | Parquet = Field(discriminator="type")
     options: dict[str, Any]
 
 
@@ -42,7 +42,7 @@ class ReadFileTransferTarget(BaseModel):
 # classes are identical but may change in the future
 class CreateFileTransferSource(BaseModel):
     directory_path: str
-    file_format: CSV | JSONLine | JSON | Excel | XML | ORC | Parquet = Field(..., discriminator="type")
+    file_format: CSV | JSONLine | JSON | Excel | XML | ORC | Parquet = Field(discriminator="type")
     options: dict[str, Any] = Field(default_factory=dict)
 
     model_config = ConfigDict(arbitrary_types_allowed=True)

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/transfers/transformations/dataframe_columns_filter.py

@@ -30,4 +30,4 @@ ColumnsFilter = IncludeFilter | RenameFilter | CastFilter
 
 class DataframeColumnsFilter(BaseModel):
     type: DATAFRAME_COLUMNS_FILTER
-    filters: list[Annotated[ColumnsFilter, Field(..., discriminator="type")]] = Field(default_factory=list)
+    filters: list[Annotated[ColumnsFilter, Field(discriminator="type")]] = Field(default_factory=list)

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/transfers/transformations/dataframe_rows_filter.py

@@ -93,4 +93,4 @@ RowsFilter = (
 
 class DataframeRowsFilter(BaseModel):
     type: DATAFRAME_ROWS_FILTER
-    filters: list[Annotated[RowsFilter, Field(..., discriminator="type")]] = Field(default_factory=list)
+    filters: list[Annotated[RowsFilter, Field(discriminator="type")]] = Field(default_factory=list)

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/transfers/transformations/file_metadata_filter.py

@@ -50,4 +50,4 @@ MetadataFilter = NameGlobFilter | NameRegexpFilter | FileSizeMinFilter | FileSiz
 
 class FileMetadataFilter(BaseModel):
     type: FILE_METADATA_FILTER
-    filters: list[Annotated[MetadataFilter, Field(..., discriminator="type")]] = Field(default_factory=list)
+    filters: list[Annotated[MetadataFilter, Field(discriminator="type")]] = Field(default_factory=list)

data_syncmaster-0.2.3/syncmaster/schemas/v1/types.py

@@ -0,0 +1,7 @@
+# SPDX-FileCopyrightText: 2023-2024 MTS PJSC
+# SPDX-License-Identifier: Apache-2.0
+from typing import Annotated
+
+from pydantic import StringConstraints
+
+NameConstr = Annotated[str, StringConstraints(min_length=3, max_length=128)]  # noqa: WPS432

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/schemas/v1/users.py

@@ -1,16 +1,12 @@
 # SPDX-FileCopyrightText: 2023-2024 MTS PJSC
 # SPDX-License-Identifier: Apache-2.0
-from pydantic import BaseModel, ConfigDict, constr
+from pydantic import BaseModel, ConfigDict
 
 from syncmaster.db.models import GroupMemberRole
 from syncmaster.db.utils import Pagination
 from syncmaster.schemas.v1.page import MetaPageSchema, PageSchema
 
 
-class UpdateUserSchema(BaseModel):
-    username: constr(pattern=r"^[_a-z0-9]+$")  # noqa: F722
-
-
 class ReadGroupMember(BaseModel):
     id: int
     username: str

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/server/api/v1/groups.py

@@ -92,7 +92,7 @@ async def read_group(
     )
 
 
-@router.patch("/groups/{group_id}")
+@router.put("/groups/{group_id}")
 async def update_group(  # noqa: WPS217
     group_id: int,
     group_data: UpdateGroupSchema,
@@ -173,7 +173,7 @@ async def read_group_users(
     return UserPageSchemaAsGroupMember.from_pagination(pagination=pagination)
 
 
-@router.patch("/groups/{group_id}/users/{user_id}")
+@router.put("/groups/{group_id}/users/{user_id}")
 async def update_user_role_group(
     group_id: int,
     user_id: int,

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/server/api/v1/queue.py

@@ -94,7 +94,7 @@ async def create_queue(
     return ReadQueueSchema.model_validate(queue, from_attributes=True)
 
 
-@router.patch("/queues/{queue_id}", description="Updating queue information")
+@router.put("/queues/{queue_id}", description="Updating queue information")
 async def update_queue(
     queue_id: int,
     queue_data: UpdateQueueSchema,

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/server/handler.py

@@ -25,7 +25,6 @@ from syncmaster.exceptions.group import (
     AlreadyIsGroupMemberError,
     AlreadyIsGroupOwnerError,
     AlreadyIsNotGroupMemberError,
-    GroupAdminNotFoundError,
     GroupAlreadyExistsError,
     GroupNotFoundError,
 )
@@ -165,13 +164,6 @@ async def syncmsater_exception_handler(request: Request, exc: SyncmasterError):
             content=content,
         )
 
-    if isinstance(exc, GroupAdminNotFoundError):
-        content.code = "not_found"
-        content.message = "Admin not found"
-        return exception_json_response(
-            status=status.HTTP_404_NOT_FOUND,
-            content=content,
-        )
     if isinstance(exc, GroupAlreadyExistsError):
         content.code = "conflict"
         content.message = "Group name already taken"

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/server/settings/auth/keycloak.py

@@ -5,11 +5,11 @@ from pydantic import BaseModel, Field, SecretStr
 
 class KeycloakSettings(BaseModel):
 
-    server_url: str = Field(..., description="Keycloak server URL")
-    client_id: str = Field(..., description="Keycloak client ID")
-    realm_name: str = Field(..., description="Keycloak realm name")
-    client_secret: SecretStr = Field(..., description="Keycloak client secret")
-    redirect_uri: str = Field(..., description="Redirect URI")
+    server_url: str = Field(description="Keycloak server URL")
+    client_id: str = Field(description="Keycloak client ID")
+    realm_name: str = Field(description="Keycloak realm name")
+    client_secret: SecretStr = Field(description="Keycloak client secret")
+    redirect_uri: str = Field(description="Redirect URI")
     verify_ssl: bool = Field(True, description="Verify SSL certificates")
     scope: str = Field("openid", description="Keycloak scope")
 

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/server/settings/server/cors.py

@@ -2,7 +2,10 @@
 # SPDX-License-Identifier: Apache-2.0
 
 
-from pydantic import BaseModel, ConfigDict, Field
+import json
+from typing import Any
+
+from pydantic import BaseModel, ConfigDict, Field, field_validator
 
 
 class CORSSettings(BaseModel):
@@ -23,20 +26,20 @@ class CORSSettings(BaseModel):
     .. code-block:: bash
 
         SYNCMASTER__SERVER__CORS__ENABLED=True
-        SYNCMASTER__SERVER__CORS__ALLOW_ORIGINS=["*"]
-        SYNCMASTER__SERVER__CORS__ALLOW_METHODS=["*"]
-        SYNCMASTER__SERVER__CORS__ALLOW_HEADERS=["*"]
-        SYNCMASTER__SERVER__CORS__EXPOSE_HEADERS=["X-Request-ID"]
+        SYNCMASTER__SERVER__CORS__ALLOW_ORIGINS="*"
+        SYNCMASTER__SERVER__CORS__ALLOW_METHODS="*"
+        SYNCMASTER__SERVER__CORS__ALLOW_HEADERS="*"
+        SYNCMASTER__SERVER__CORS__EXPOSE_HEADERS=X-Request-ID,Location,Access-Control-Allow-Credentials
 
     For production environment:
 
     .. code-block:: bash
 
         SYNCMASTER__SERVER__CORS__ENABLED=True
-        SYNCMASTER__SERVER__CORS__ALLOW_ORIGINS=["production.example.com"]
-        SYNCMASTER__SERVER__CORS__ALLOW_METHODS=["GET"]
-        SYNCMASTER__SERVER__CORS__ALLOW_HEADERS=["X-Request-ID", "X-Request-With"]
-        SYNCMASTER__SERVER__CORS__EXPOSE_HEADERS=["X-Request-ID"]
+        SYNCMASTER__SERVER__CORS__ALLOW_ORIGINS="production.example.com"
+        SYNCMASTER__SERVER__CORS__ALLOW_METHODS="GET"
+        SYNCMASTER__SERVER__CORS__ALLOW_HEADERS="X-Request-ID,X-Request-With"
+        SYNCMASTER__SERVER__CORS__EXPOSE_HEADERS="X-Request-ID"
         # custom option passed directly to middleware
         SYNCMASTER__SERVER__CORS__MAX_AGE=600
     """
@@ -55,4 +58,13 @@ class CORSSettings(BaseModel):
     )
     expose_headers: list[str] = Field(default=["X-Request-ID"], description="HTTP headers exposed from server")
 
+    @field_validator("allow_origins", "allow_methods", "allow_headers", "expose_headers", mode="before")
+    @classmethod
+    def _validate_bootstrap_servers(cls, value: Any):
+        if not isinstance(value, str):
+            return value
+        if "[" in value:
+            return json.loads(value)
+        return [item.strip() for item in value.split(",")]
+
     model_config = ConfigDict(extra="allow")

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/settings/log/colored.yml

@@ -40,7 +40,7 @@ loggers:
     level: INFO
     propagate: false
   celery:
-    level: DEBUG
+    level: INFO
     handlers: [celery]
     propagate: false
   scheduler:
@@ -51,3 +51,7 @@ loggers:
     handlers: [main]
     level: WARNING
     propagate: false
+  hdfs.client:
+    handlers: [main]
+    level: WARNING
+    propagate: false

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/settings/log/json.yml

@@ -39,7 +39,7 @@ loggers:
     level: INFO
     propagate: false
   celery:
-    level: DEBUG
+    level: INFO
     handlers: [celery]
     propagate: false
   scheduler:
@@ -50,3 +50,7 @@ loggers:
     handlers: [main]
     level: WARNING
     propagate: false
+  hdfs.client:
+    handlers: [main]
+    level: WARNING
+    propagate: false

{data_syncmaster-0.2.1 → data_syncmaster-0.2.3}/syncmaster/settings/log/plain.yml

@@ -40,7 +40,7 @@ loggers:
     level: INFO
     propagate: false
   celery:
-    level: DEBUG
+    level: INFO
     handlers: [celery]
     propagate: false
   scheduler:
@@ -51,3 +51,7 @@ loggers:
     handlers: [main]
     level: WARNING
     propagate: false
+  hdfs.client:
+    handlers: [main]
+    level: WARNING
+    propagate: false