From fe932fee19d73cc8005c9c94e93d480c18d31bb6 Mon Sep 17 00:00:00 2001 From: dudulu <50397689+hgalytoby@users.noreply.github.com> Date: Wed, 26 Jul 2023 19:10:03 +0800 Subject: [PATCH] Add request parameter to delete user method (#1258) * add request parameter to delete_user method. * Docs: Add request parameter to docstring --- fastapi_users/manager.py | 2 ++ fastapi_users/router/users.py | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/fastapi_users/manager.py b/fastapi_users/manager.py index 60eb6c32..85ca0261 100644 --- a/fastapi_users/manager.py +++ b/fastapi_users/manager.py @@ -475,6 +475,8 @@ class BaseUserManager(Generic[models.UP, models.ID]): Delete a user. :param user: The user to delete. + :param request: Optional FastAPI request that + triggered the operation, defaults to None. """ await self.on_before_delete(user, request) await self.user_db.delete(user) diff --git a/fastapi_users/router/users.py b/fastapi_users/router/users.py index 19e04066..b3cc4351 100644 --- a/fastapi_users/router/users.py +++ b/fastapi_users/router/users.py @@ -217,10 +217,11 @@ def get_users_router( }, ) async def delete_user( + request: Request, user=Depends(get_user_or_404), user_manager: BaseUserManager[models.UP, models.ID] = Depends(get_user_manager), ): - await user_manager.delete(user) + await user_manager.delete(user, request=request) return None return router