Status code when user exists

Clinical-Genomics · Oct 21, 2021 · 42f7a01 · 42f7a01
1 parent 8799d94
commit 42f7a01
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/statina/API/external/api/deps.py b/statina/API/external/api/deps.py
@@ -58,9 +58,7 @@ def find_user(username: str) -> Optional[User]:
     adapter: StatinaAdapter = get_nipt_adapter()
     user: User = find.user(adapter=adapter, user_name=username)
 
-    if user and user.role != "inactive":
-        return user
-    return None
+    return user
 
 
 def get_user_scopes(username: str) -> list:

diff --git a/statina/API/v2/endpoints/user.py b/statina/API/v2/endpoints/user.py
@@ -106,6 +106,9 @@ async def register_user(
     password: str = Form(...),
     adapter: StatinaAdapter = Depends(get_nipt_adapter),
 ):
+    user = find_user(username=username)
+    if user:
+        return JSONResponse(f"Username {username} already taken!", status_code=409)
     if not secrets.compare_digest(password, password_repeated):
         return JSONResponse(content="Password mismatch!", status_code=400)