Закрыть публичную регистрацию. Создание пользователей — только через CLI или админа.
This commit is contained in:
@@ -1,4 +1,4 @@
|
|||||||
from fastapi import APIRouter, Depends, HTTPException
|
from fastapi import APIRouter, Depends, HTTPException, status
|
||||||
from fastapi.security import OAuth2PasswordRequestForm
|
from fastapi.security import OAuth2PasswordRequestForm
|
||||||
from sqlalchemy.orm import Session
|
from sqlalchemy.orm import Session
|
||||||
|
|
||||||
@@ -12,28 +12,43 @@ router = APIRouter(prefix="/auth", tags=["auth"])
|
|||||||
|
|
||||||
@router.post("/register", response_model=schemas.UserOut)
|
@router.post("/register", response_model=schemas.UserOut)
|
||||||
def register(payload: schemas.UserCreate, db: Session = Depends(get_db)):
|
def register(payload: schemas.UserCreate, db: Session = Depends(get_db)):
|
||||||
existing = db.query(models.User).filter(models.User.email == payload.email).first()
|
"""Регистрация закрыта для публичного доступа.
|
||||||
if existing:
|
|
||||||
raise HTTPException(status_code=400, detail="Пользователь с таким email уже существует")
|
|
||||||
|
|
||||||
# первый зарегистрированный пользователь становится администратором
|
Единственное исключение — bootstrap первого администратора: когда в БД
|
||||||
is_first_user = db.query(models.User).count() == 0
|
ещё нет ни одного пользователя, регистрация создаёт admin-аккаунт. Это
|
||||||
|
нужно, чтобы панель можно было поднять с нуля. Все последующие
|
||||||
|
пользователи создаются только через CLI (`python -m app.manage_users`)
|
||||||
|
или админский эндпоинт /admin/users.
|
||||||
|
"""
|
||||||
|
email = payload.email.lower().strip()
|
||||||
|
|
||||||
|
# Bootstrap: первый пользователь становится администратором.
|
||||||
|
if db.query(models.User).count() == 0:
|
||||||
user = models.User(
|
user = models.User(
|
||||||
email=payload.email,
|
email=email,
|
||||||
password_hash=hash_password(payload.password),
|
password_hash=hash_password(payload.password),
|
||||||
role=models.Role.admin if is_first_user else models.Role.client,
|
role=models.Role.admin,
|
||||||
|
is_active=True,
|
||||||
)
|
)
|
||||||
db.add(user)
|
db.add(user)
|
||||||
db.commit()
|
db.commit()
|
||||||
db.refresh(user)
|
db.refresh(user)
|
||||||
return user
|
return user
|
||||||
|
|
||||||
|
# Публичная регистрация закрыта — отвечаем 403.
|
||||||
|
raise HTTPException(
|
||||||
|
status_code=status.HTTP_403_FORBIDDEN,
|
||||||
|
detail="Регистрация закрыта. Обратитесь к администратору для создания учётной записи.",
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
@router.post("/login", response_model=schemas.Token)
|
@router.post("/login", response_model=schemas.Token)
|
||||||
def login(form_data: OAuth2PasswordRequestForm = Depends(), db: Session = Depends(get_db)):
|
def login(form_data: OAuth2PasswordRequestForm = Depends(), db: Session = Depends(get_db)):
|
||||||
user = db.query(models.User).filter(models.User.email == form_data.username).first()
|
user = db.query(models.User).filter(models.User.email == form_data.username).first()
|
||||||
if not user or not verify_password(form_data.password, user.password_hash):
|
if not user or not verify_password(form_data.password, user.password_hash):
|
||||||
raise HTTPException(status_code=401, detail="Неверный email или пароль")
|
raise HTTPException(status_code=401, detail="Неверный email или пароль")
|
||||||
|
if not user.is_active:
|
||||||
|
raise HTTPException(status_code=403, detail="Аккаунт деактивирован. Обратитесь к администратору.")
|
||||||
token = create_access_token({"sub": str(user.id)})
|
token = create_access_token({"sub": str(user.id)})
|
||||||
return {"access_token": token, "token_type": "bearer"}
|
return {"access_token": token, "token_type": "bearer"}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user