Skip to content

Commit 9e3c9fa

Browse files
committed
some rename
1 parent e3ffe7d commit 9e3c9fa

File tree

7 files changed

+38
-33
lines changed

7 files changed

+38
-33
lines changed

api/specs/web-server/_users.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@
88

99
from fastapi import APIRouter, Depends, status
1010
from models_library.api_schemas_webserver.users import (
11+
MyPermissionGet,
1112
MyProfileGet,
1213
MyProfilePatch,
13-
PermissionGet,
14-
ThirdPartyToken,
15-
TokenCreate,
14+
MyTokenCreate,
1615
UserGet,
1716
UsersSearchQueryParams,
17+
UserThirdPartyToken,
1818
)
1919
from models_library.api_schemas_webserver.users_preferences import PatchRequestBody
2020
from models_library.generics import Envelope
@@ -73,24 +73,24 @@ async def set_frontend_preference(
7373

7474
@router.get(
7575
"/me/tokens",
76-
response_model=Envelope[list[ThirdPartyToken]],
76+
response_model=Envelope[list[UserThirdPartyToken]],
7777
)
7878
async def list_tokens():
7979
...
8080

8181

8282
@router.post(
8383
"/me/tokens",
84-
response_model=Envelope[ThirdPartyToken],
84+
response_model=Envelope[UserThirdPartyToken],
8585
status_code=status.HTTP_201_CREATED,
8686
)
87-
async def create_token(_token: TokenCreate):
87+
async def create_token(_token: MyTokenCreate):
8888
...
8989

9090

9191
@router.get(
9292
"/me/tokens/{service}",
93-
response_model=Envelope[ThirdPartyToken],
93+
response_model=Envelope[UserThirdPartyToken],
9494
)
9595
async def get_token(_params: Annotated[_TokenPathParams, Depends()]):
9696
...
@@ -133,7 +133,7 @@ async def mark_notification_as_read(
133133

134134
@router.get(
135135
"/me/permissions",
136-
response_model=Envelope[list[PermissionGet]],
136+
response_model=Envelope[list[MyPermissionGet]],
137137
)
138138
async def list_user_permissions():
139139
...

packages/models-library/src/models_library/api_schemas_webserver/users.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
#
2121
# THIRD-PARTY TOKENS
2222
#
23-
class ThirdPartyToken(BaseModel):
23+
class UserThirdPartyToken(BaseModel):
2424
"""
2525
Tokens used to access third-party services connected to osparc (e.g. pennsieve, scicrunch, etc)
2626
"""
@@ -42,19 +42,19 @@ class ThirdPartyToken(BaseModel):
4242
)
4343

4444

45-
class TokenCreate(ThirdPartyToken):
45+
class MyTokenCreate(UserThirdPartyToken):
4646
...
4747

4848

4949
#
5050
# PERMISSIONS
5151
#
52-
class Permission(BaseModel):
52+
class UserPermission(BaseModel):
5353
name: str
5454
allowed: bool
5555

5656

57-
class PermissionGet(Permission, OutputSchema):
57+
class MyPermissionGet(UserPermission, OutputSchema):
5858
...
5959

6060

services/web/server/src/simcore_service_webserver/users/_notifications_handlers.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
import redis.asyncio as aioredis
55
from aiohttp import web
6-
from models_library.api_schemas_webserver.users import Permission, PermissionGet
6+
from models_library.api_schemas_webserver.users import MyPermissionGet, UserPermission
77
from pydantic import BaseModel
88
from servicelib.aiohttp import status
99
from servicelib.aiohttp.requests_validation import (
@@ -125,12 +125,12 @@ async def mark_notification_as_read(request: web.Request) -> web.Response:
125125
@permission_required("user.permissions.read")
126126
async def list_user_permissions(request: web.Request) -> web.Response:
127127
req_ctx = UsersRequestContext.model_validate(request)
128-
list_permissions: list[Permission] = await _users_service.list_user_permissions(
128+
list_permissions: list[UserPermission] = await _users_service.list_user_permissions(
129129
request.app, user_id=req_ctx.user_id, product_name=req_ctx.product_name
130130
)
131131
return envelope_json_response(
132132
[
133-
PermissionGet.model_construct(
133+
MyPermissionGet.model_construct(
134134
_fields_set=p.model_fields_set, **p.model_dump()
135135
)
136136
for p in list_permissions

services/web/server/src/simcore_service_webserver/users/_tokens.py

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,10 @@
44
"""
55
import sqlalchemy as sa
66
from aiohttp import web
7-
from models_library.api_schemas_webserver.users import ThirdPartyToken, TokenCreate
7+
from models_library.api_schemas_webserver.users import (
8+
MyTokenCreate,
9+
UserThirdPartyToken,
10+
)
811
from models_library.users import UserID
912
from models_library.utils.fastapi_encoders import jsonable_encoder
1013
from sqlalchemy import and_, literal_column
@@ -15,8 +18,8 @@
1518

1619

1720
async def create_token(
18-
app: web.Application, user_id: UserID, token: TokenCreate
19-
) -> ThirdPartyToken:
21+
app: web.Application, user_id: UserID, token: MyTokenCreate
22+
) -> UserThirdPartyToken:
2023
async with get_database_engine(app).acquire() as conn:
2124
await conn.execute(
2225
tokens.insert().values(
@@ -28,33 +31,35 @@ async def create_token(
2831
return token
2932

3033

31-
async def list_tokens(app: web.Application, user_id: UserID) -> list[ThirdPartyToken]:
32-
user_tokens: list[ThirdPartyToken] = []
34+
async def list_tokens(
35+
app: web.Application, user_id: UserID
36+
) -> list[UserThirdPartyToken]:
37+
user_tokens: list[UserThirdPartyToken] = []
3338
async with get_database_engine(app).acquire() as conn:
3439
async for row in conn.execute(
3540
sa.select(tokens.c.token_data).where(tokens.c.user_id == user_id)
3641
):
37-
user_tokens.append(ThirdPartyToken.model_construct(**row["token_data"]))
42+
user_tokens.append(UserThirdPartyToken.model_construct(**row["token_data"]))
3843
return user_tokens
3944

4045

4146
async def get_token(
4247
app: web.Application, user_id: UserID, service_id: str
43-
) -> ThirdPartyToken:
48+
) -> UserThirdPartyToken:
4449
async with get_database_engine(app).acquire() as conn:
4550
result = await conn.execute(
4651
sa.select(tokens.c.token_data).where(
4752
and_(tokens.c.user_id == user_id, tokens.c.token_service == service_id)
4853
)
4954
)
5055
if row := await result.first():
51-
return ThirdPartyToken.model_construct(**row["token_data"])
56+
return UserThirdPartyToken.model_construct(**row["token_data"])
5257
raise TokenNotFoundError(service_id=service_id)
5358

5459

5560
async def update_token(
5661
app: web.Application, user_id: UserID, service_id: str, token_data: dict[str, str]
57-
) -> ThirdPartyToken:
62+
) -> UserThirdPartyToken:
5863
async with get_database_engine(app).acquire() as conn:
5964
result = await conn.execute(
6065
sa.select(tokens.c.token_data, tokens.c.token_id).where(
@@ -78,7 +83,7 @@ async def update_token(
7883
assert resp.rowcount == 1 # nosec
7984
updated_token = await resp.fetchone()
8085
assert updated_token # nosec
81-
return ThirdPartyToken.model_construct(**updated_token["token_data"])
86+
return UserThirdPartyToken.model_construct(**updated_token["token_data"])
8287

8388

8489
async def delete_token(app: web.Application, user_id: UserID, service_id: str) -> None:

services/web/server/src/simcore_service_webserver/users/_tokens_handlers.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
import logging
33

44
from aiohttp import web
5-
from models_library.api_schemas_webserver.users import TokenCreate
5+
from models_library.api_schemas_webserver.users import MyTokenCreate
66
from pydantic import BaseModel
77
from servicelib.aiohttp import status
88
from servicelib.aiohttp.requests_validation import (
@@ -55,7 +55,7 @@ async def list_tokens(request: web.Request) -> web.Response:
5555
@permission_required("user.tokens.*")
5656
async def create_token(request: web.Request) -> web.Response:
5757
req_ctx = UsersRequestContext.model_validate(request)
58-
token_create = await parse_request_body_as(TokenCreate, request)
58+
token_create = await parse_request_body_as(MyTokenCreate, request)
5959
await _tokens.create_token(request.app, req_ctx.user_id, token_create)
6060
return envelope_json_response(token_create, web.HTTPCreated)
6161

services/web/server/src/simcore_service_webserver/users/_users_repository.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import sqlalchemy as sa
44
from aiohttp import web
5-
from models_library.api_schemas_webserver.users import Permission
5+
from models_library.api_schemas_webserver.users import UserPermission
66
from models_library.users import GroupID, UserBillingDetails, UserID
77
from simcore_postgres_database.models.groups import groups, user_to_groups
88
from simcore_postgres_database.models.products import products
@@ -73,8 +73,8 @@ async def list_user_permissions(
7373
*,
7474
user_id: UserID,
7575
product_name: str,
76-
) -> list[Permission]:
77-
override_services_specifications = Permission(
76+
) -> list[UserPermission]:
77+
override_services_specifications = UserPermission(
7878
name="override_services_specifications",
7979
allowed=False,
8080
)

services/web/server/src/simcore_service_webserver/users/_users_service.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import pycountry
44
from aiohttp import web
5-
from models_library.api_schemas_webserver.users import Permission, UserGet
5+
from models_library.api_schemas_webserver.users import UserGet, UserPermission
66
from models_library.emails import LowerCaseEmailStr
77
from models_library.payments import UserInvoiceAddress
88
from models_library.products import ProductName
@@ -24,8 +24,8 @@ async def list_user_permissions(
2424
*,
2525
user_id: UserID,
2626
product_name: ProductName,
27-
) -> list[Permission]:
28-
permissions: list[Permission] = await _users_repository.list_user_permissions(
27+
) -> list[UserPermission]:
28+
permissions: list[UserPermission] = await _users_repository.list_user_permissions(
2929
app, user_id=user_id, product_name=product_name
3030
)
3131
return permissions

0 commit comments

Comments
 (0)