🔧 chore(users.py): remove unused current_user variable in read_all_users function

🔧 chore(api_key.py): add cascade delete to user-api_key relationship to ensure API keys are deleted when user is deleted
This commit is contained in:
Gabriel Luiz Freitas Almeida 2023-09-25 18:06:52 -03:00
commit f497369a1c
2 changed files with 6 additions and 2 deletions

View file

@ -67,7 +67,7 @@ def read_current_user(
def read_all_users(
skip: int = 0,
limit: int = 10,
current_user: Session = Depends(get_current_active_superuser),
_: Session = Depends(get_current_active_superuser),
session: Session = Depends(get_session),
) -> UsersResponse:
"""

View file

@ -22,8 +22,12 @@ class ApiKey(ApiKeyBase, table=True):
api_key: str = Field(index=True, unique=True)
# User relationship
# Delete API keys when user is deleted
user_id: UUID = Field(index=True, foreign_key="user.id")
user: "User" = Relationship(back_populates="api_keys")
user: "User" = Relationship(
back_populates="api_keys",
sa_relationship_kwargs={"cascade": "delete"},
)
class ApiKeyCreate(ApiKeyBase):