Files
anti-plagiarism/services/api/app/core/security.py
jze9 61c903ae0e
All checks were successful
Deploy / test (push) Successful in 3m26s
Deploy / deploy (push) Successful in 1m41s
fix(api): 500 на GET /tasks/{id} при обращении к закэшированному юзеру
Живая проверка после первого реального 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>
2026-08-24 18:31:20 +05:00

163 lines
6.1 KiB
Python
Raw Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
"""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}")