patrickandClaude Sonnet 5
ceddeb1bf4
Konsolidierte Funde aus postgres-/sql-/jwt-/owasp-top10-expert-Review: - Race Conditions gefixt: doppelte aktive Kontrolle (SAVEPOINT + partieller Unique-Index), doppelte Mindermengen-Genehmigung (FOR UPDATE + Unique-Index), Lost-Update bei Nachfüllung (FOR UPDATE auf Fehlbestand/Objektposition). - Migration 0007: partielle Unique-Indizes als DB-Sicherheitsnetz + fehlende FK-Indizes (fehlbestand.material_id, kontrolle(objekt_id,status), zustaendigkeit, benutzer_rolle.rolle, objektposition.ablaufdatum u.a.). - Connection-Pool explizit begrenzt (pool_size=5, max_overflow=5) - ohne das könnte jeder uvicorn-Worker den Postgres max_connections-Wert sprengen. - Timing-Angriff bei Login-Enumeration gefixt (konstante Antwortzeit über Dummy-Hash), JWT-Decode verlangt jetzt exp/sub-Claims. - App-seitiges Rate-Limiting (slowapi, 5/min) auf /auth/login als Verteidigung in der Tiefe zusätzlich zum nginx-Limit. - nginx: TLS mit selbstsigniertem Zertifikat (LAN-Betrieb, keine Domain), HSTS, Content-Security-Policy, Permissions-Policy ergänzt. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CVgbozhYmuEhiEJHffRXCV
36 lines
1.3 KiB
Python
36 lines
1.3 KiB
Python
from collections.abc import AsyncGenerator
|
|
|
|
from sqlalchemy.ext.asyncio import AsyncSession, async_sessionmaker, create_async_engine
|
|
|
|
from app.core.app_settings import settings
|
|
|
|
# Explizite Pool-Grenzen (postgres-expert-Review-Fund): ohne das erzeugt jeder
|
|
# uvicorn-Worker einen eigenen Default-Pool (5 + 10 Overflow), was bei mehreren
|
|
# Workern auf dem 4GB-VPS PostgreSQL max_connections (per Tuning-Config auf 50
|
|
# begrenzt, siehe deploy/install_server.sh) sprengen kann. pool_recycle gegen
|
|
# von der DB nach Idle-Timeout gekappte Verbindungen.
|
|
engine = create_async_engine(
|
|
settings.database_url,
|
|
pool_pre_ping=True,
|
|
pool_size=5,
|
|
max_overflow=5,
|
|
pool_recycle=1800,
|
|
)
|
|
SessionLocal = async_sessionmaker(engine, expire_on_commit=False)
|
|
|
|
|
|
async def get_db() -> AsyncGenerator[AsyncSession, None]:
|
|
"""Konvention für alle Endpunkte: Erfolg -> commit, Exception -> rollback.
|
|
|
|
Endpunkte selbst rufen kein `db.commit()`/`db.rollback()` auf, sondern verlassen
|
|
sich auf diese Dependency (Review-Fund: Konvention muss zentral, nicht pro
|
|
Endpunkt copy-paste sein).
|
|
"""
|
|
async with SessionLocal() as session:
|
|
try:
|
|
yield session
|
|
await session.commit()
|
|
except Exception:
|
|
await session.rollback()
|
|
raise
|