diff --git a/CODE_AUDIT.md b/CODE_AUDIT.md index 8261ddd..b153b66 100644 --- a/CODE_AUDIT.md +++ b/CODE_AUDIT.md @@ -200,7 +200,7 @@ entries), problématique au-delà. **Recommandation** : batcher les déchiffrements (Promise.all par lots), et si besoin paginer côté serveur pour la vue grille. -### 2.5 🟡 500 au lieu de 401 sur session expirée +### 2.5 ✅ 500 au lieu de 401 sur session expirée — corrigé (2026-07-09) `Authenticate` écrit 401 puis `raise ESessionRejected` → le `try/except` global de `PM.HTTPServer` réécrit un **500**. Pré-existant, tous les diff --git a/delphi-backend/Source/PM.HTTPServer.pas b/delphi-backend/Source/PM.HTTPServer.pas index b119f19..7e39a7b 100644 --- a/delphi-backend/Source/PM.HTTPServer.pas +++ b/delphi-backend/Source/PM.HTTPServer.pas @@ -15,7 +15,7 @@ uses Winapi.Windows, IdHTTPServer, IdContext, IdCustomHTTPServer, IdSocketHandle, IdTCPConnection, PM.Router, PM.JSON, PM.Database, PM.StaticFiles, PM.EmbeddedAssets, - PM.Crypto, PM.ProcessLockdown; + PM.Crypto, PM.ProcessLockdown, PM.Session; type TLogProc = reference to procedure(const AMsg: string); @@ -284,6 +284,9 @@ begin if Assigned(StaticServer) and StaticServer.TryServe(ARequest, AResponse) then Exit; TJSONHelper.SendError(AResponse, 404, 'Not found'); except + // Authenticate/RequireCSRF already wrote the 401 — don't overwrite it + // with a 500. Any handler that doesn't wrap Authenticate lands here. + on ESessionRejected do Exit; on E: Exception do begin Log('ERROR ' + ARequest.Command + ' ' + ARequest.Document + ' : ' + E.Message); @@ -308,6 +311,7 @@ begin if not Router.DispatchRequest(ARequest, AResponse) then TJSONHelper.SendError(AResponse, 404, 'Not found'); except + on ESessionRejected do Exit; // 401 already sent — keep it, don't 500 on E: Exception do begin Log('ERROR ' + ARequest.Command + ' ' + ARequest.Document + ' : ' + E.Message);