From ce4cfc3240e8c106f5eb02698dbaee8c8e650c56 Mon Sep 17 00:00:00 2001 From: chihlasm Date: Fri, 10 Apr 2026 07:12:45 +0000 Subject: [PATCH] fix: set account_id on PsaPostLog in psa_post_to_ticket (missed third write path); fix get_admin_db docstring Co-Authored-By: Claude Sonnet 4.6 --- backend/app/api/endpoints/sessions.py | 1 + backend/app/core/admin_database.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/backend/app/api/endpoints/sessions.py b/backend/app/api/endpoints/sessions.py index 3ad51f5d..3dee5085 100644 --- a/backend/app/api/endpoints/sessions.py +++ b/backend/app/api/endpoints/sessions.py @@ -1102,6 +1102,7 @@ async def psa_post_to_ticket( # Log to audit trail log_entry = PsaPostLog( session_id=session.id, + account_id=session.account_id, psa_connection_id=psa_connection.id if psa_connection else None, ticket_id=session.psa_ticket_id, note_type=data.note_type, diff --git a/backend/app/core/admin_database.py b/backend/app/core/admin_database.py index 26a5b7f7..9d845c34 100644 --- a/backend/app/core/admin_database.py +++ b/backend/app/core/admin_database.py @@ -27,7 +27,7 @@ _admin_session_factory = async_sessionmaker( async def get_admin_db() -> AsyncGenerator[AsyncSession, None]: - """Yield an admin DB session (BYPASSRLS). Use only on /admin/* endpoints.""" + """Yield an admin DB session (BYPASSRLS). See module docstring for approved use cases.""" async with _admin_session_factory() as session: try: yield session