Живая проверка после первого реального OAuth-логина: юзер вошёл, поиск
отработал, но статус задачи не показывался — GET /api/tasks/{id} падал 500.
В логе: AttributeError: 'NoneType' object has no attribute
'supports_population' на current_user.id.
Причина: _load_user() при попадании в Redis-кэш делал
User.__new__(User); u.__dict__.update(data) — выглядело как лёгкий объект
без лишнего SELECT, но замапленные атрибуты User (id, email, ...) —
дескрипторы данных SQLAlchemy: их __get__ обращается к InstanceState,
которого у объекта в обход __init__/ORM-машинерии нет. Падало на КАЖДОМ
запросе, где юзер брался из кэша (5 мин TTL) — то есть почти всегда,
кроме первого запроса после логина/протухания кэша.
Фикс: CachedUser — обычный dataclass с теми же полями, без дескрипторов,
падать нечему. Заодно нашёл тем же грепом идентичный баг в
resend_verification (трогал current_user.verification_token — не входит
в кэшируемый набор полей, к тому же current_user из кэша не привязан к
сессии — db.commit()/refresh() на нём тоже не сработали бы) — почистил по
образцу update_profile/change_password: подгружает свежего юзера из БД.
Подтверждено вручную на проде: сгенерировал JWT для реального юзера,
GET /api/tasks/{id} возвращал 500 до фикса.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
163 lines
6.1 KiB
Python
163 lines
6.1 KiB
Python
"""JWT, хэширование паролей, dependency получения пользователя с кэшированием."""
|
||
|
||
import json
|
||
import logging
|
||
from dataclasses import dataclass
|
||
from datetime import UTC, datetime, timedelta
|
||
from typing import Any
|
||
|
||
from fastapi import Depends, HTTPException, Query, WebSocket, status
|
||
from fastapi.security import OAuth2PasswordBearer
|
||
from jose import JWTError, jwt
|
||
from passlib.context import CryptContext
|
||
from sqlalchemy import select
|
||
from sqlalchemy.ext.asyncio import AsyncSession
|
||
|
||
from app.config import settings
|
||
from app.core.redis_client import get_redis
|
||
from app.database import get_db
|
||
|
||
logger = logging.getLogger(__name__)
|
||
|
||
pwd_context = CryptContext(schemes=["bcrypt"], deprecated="auto")
|
||
oauth2_scheme = OAuth2PasswordBearer(tokenUrl="/api/auth/login")
|
||
|
||
# TTL кэша пользователя в Redis (5 минут)
|
||
_USER_CACHE_TTL = 300
|
||
|
||
|
||
def hash_password(password: str) -> str:
|
||
return pwd_context.hash(password)
|
||
|
||
|
||
def verify_password(plain: str, hashed: str | None) -> bool:
|
||
# OAuth-пользователи не имеют пароля (hashed_password=None) — попытка
|
||
# войти паролем должна отвечать "неверный пароль", а не падать 500-й.
|
||
if hashed is None:
|
||
return False
|
||
return pwd_context.verify(plain, hashed)
|
||
|
||
|
||
def create_access_token(data: dict[str, Any], expires_delta: timedelta | None = None) -> str:
|
||
payload = data.copy()
|
||
expire = datetime.now(UTC) + (
|
||
expires_delta or timedelta(minutes=settings.ACCESS_TOKEN_EXPIRE_MINUTES)
|
||
)
|
||
payload["exp"] = expire
|
||
return jwt.encode(payload, settings.SECRET_KEY, algorithm=settings.ALGORITHM)
|
||
|
||
|
||
def _decode_token(token: str) -> int:
|
||
"""
|
||
Декодировать JWT и вернуть user_id.
|
||
Raises HTTPException 401 при любой проблеме с токеном.
|
||
"""
|
||
try:
|
||
payload = jwt.decode(token, settings.SECRET_KEY, algorithms=[settings.ALGORITHM])
|
||
sub = payload.get("sub")
|
||
if sub is None:
|
||
raise ValueError("missing sub")
|
||
return int(sub)
|
||
except (JWTError, ValueError):
|
||
raise HTTPException(
|
||
status_code=status.HTTP_401_UNAUTHORIZED,
|
||
detail="Невалидный или просроченный токен",
|
||
headers={"WWW-Authenticate": "Bearer"},
|
||
) from None
|
||
|
||
|
||
@dataclass
|
||
class CachedUser:
|
||
"""Пользователь из Redis-кэша — обычный dataclass, НЕ SQLAlchemy-модель.
|
||
|
||
Раньше здесь был User.__new__(User) + __dict__.update(data) — казалось
|
||
эквивалентом, но замапленные атрибуты User (id, email, ...) — дескрипторы
|
||
данных: их __get__ обращается к self.impl, который берётся из InstanceState,
|
||
а у объекта, созданного в обход __init__/ORM-машинерии, состояния нет.
|
||
Итог — AttributeError на любое обращение к полю из кэша (нашли на
|
||
GET /tasks/{id}: current_user.id падал 500-й, как только юзер брался не
|
||
из БД, а из кэша). Обычный dataclass с теми же именами полей — просто
|
||
plain-атрибуты, без дескрипторов, падать нечему.
|
||
"""
|
||
|
||
id: int
|
||
email: str
|
||
name: str
|
||
plan: str
|
||
is_verified: bool
|
||
is_admin: bool
|
||
|
||
|
||
async def _load_user(user_id: int, db: AsyncSession):
|
||
"""Загрузить пользователя из Redis-кэша или из БД."""
|
||
from app.models.user import User # избегаем circular import на уровне модуля
|
||
|
||
cache_key = f"user:cache:{user_id}"
|
||
r = get_redis()
|
||
|
||
# Пробуем кэш
|
||
cached = await r.get(cache_key)
|
||
if cached:
|
||
return CachedUser(**json.loads(cached))
|
||
|
||
# Кэш пустой — идём в БД
|
||
result = await db.execute(select(User).where(User.id == user_id))
|
||
user = result.scalar_one_or_none()
|
||
|
||
if user is None:
|
||
raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="Пользователь не найден")
|
||
|
||
# Сохраняем в кэш (только безопасные поля, без hashed_password)
|
||
safe = {
|
||
"id": user.id,
|
||
"email": user.email,
|
||
"name": user.name,
|
||
"plan": user.plan,
|
||
"is_verified": user.is_verified,
|
||
"is_admin": user.is_admin,
|
||
}
|
||
await r.setex(cache_key, _USER_CACHE_TTL, json.dumps(safe))
|
||
|
||
return user
|
||
|
||
|
||
async def get_current_user(
|
||
token: str = Depends(oauth2_scheme),
|
||
db: AsyncSession = Depends(get_db),
|
||
):
|
||
"""Dependency: текущий пользователь из JWT с Redis-кэшем."""
|
||
user_id = _decode_token(token)
|
||
return await _load_user(user_id, db)
|
||
|
||
|
||
async def get_current_verified_user(current_user=Depends(get_current_user)):
|
||
"""Dependency: только верифицированные пользователи."""
|
||
if not current_user.is_verified:
|
||
raise HTTPException(
|
||
status_code=status.HTTP_403_FORBIDDEN,
|
||
detail="Необходимо подтвердить email адрес",
|
||
)
|
||
return current_user
|
||
|
||
|
||
async def get_ws_user(
|
||
websocket: WebSocket,
|
||
token: str = Query(..., description="JWT токен (передаётся как query-параметр ?token=...)"),
|
||
db: AsyncSession = Depends(get_db),
|
||
):
|
||
"""
|
||
Dependency для WebSocket: аутентификация через query-параметр ?token=JWT.
|
||
|
||
WebSocket API браузера не позволяет передавать Authorization header,
|
||
поэтому токен передаётся в query-строке:
|
||
ws://host/ws/tasks/{public_id}?token=<JWT>
|
||
"""
|
||
user_id = _decode_token(token)
|
||
return await _load_user(user_id, db)
|
||
|
||
|
||
async def invalidate_user_cache(user_id: int) -> None:
|
||
"""Сбросить кэш пользователя (при смене тарифа, пароля и т.д.)."""
|
||
r = get_redis()
|
||
await r.delete(f"user:cache:{user_id}")
|