Skip to content

Commit

Permalink
chore: primero verifico que el usuario no esté registrado y despues l…
Browse files Browse the repository at this point in the history
…a contraseña
  • Loading branch information
fedecarboni7 committed Aug 13, 2024
1 parent 0b5de28 commit 0a95bec
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions app/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -132,17 +132,16 @@ async def signup(
# Validar nombre de usuario y contraseña
try:
validate_username(username)

user = execute_with_retries(query_user, db, username)
if user:
raise ValueError("Usuario ya registrado")

validate_password(password)
except ValueError as e:
return templates.TemplateResponse("signup.html", {"request": request, "error": str(e)})

try:
user = execute_with_retries(query_user, db, username)
except OperationalError:
return HTMLResponse("Error al acceder a la base de datos. Inténtalo de nuevo más tarde.", status_code=500)

if user:
return templates.TemplateResponse(request=request, name="signup.html", context={"error": "Usuario ya registrado"})

new_user = User(username=username)
new_user.set_password(password)
Expand Down

0 comments on commit 0a95bec

Please sign in to comment.