Segreti con default pubblico e nessun controllo all'avvio; session.permanent mai impostato #33

New Issue

2026-09-21T20:14:52Z

Adriano commented

2026-09-21 20:14:52 +00:00

GRAVITÀ: media
FILE: src/frontend/flask_app/config.py:12,22 ; src/backend/config.py:14,19 ; docker-compose.yml:7,10 ; docker-compose.dev.yml:7,10 ; src/frontend/flask_app/blueprints/auth.py:85,121
COSA: DB_ROOT_PASSWORD, DB_PASSWORD, SERVER_SECRET_KEY, CLIENT_SECRET_KEY hanno default committati; un .env incompleto parte in silenzio e le sessioni Flask diventano falsificabili (ruoli, is_admin). session.permanent mai impostato: PERMANENT_SESSION_LIFETIME=8h non ha effetto. Il cookie contiene api_key in chiaro (firmato, non cifrato). /auth/logout accetta GET.
VERIFICATO: grep -rn permanent src/frontend/flask_app/*.py blueprints → 0; grep -rn "change-this\|change_me" src --include=*.py → solo i default.

ORIGINE: revisione fable del 21/09/2026 su ramo V3.0.0, commit 8cc5e82. Documento completo: _output/2026-09-21-TieMeasureFlow-revisione.md su Wasabi.

GRAVITÀ: media FILE: src/frontend/flask_app/config.py:12,22 ; src/backend/config.py:14,19 ; docker-compose.yml:7,10 ; docker-compose.dev.yml:7,10 ; src/frontend/flask_app/blueprints/auth.py:85,121 COSA: DB_ROOT_PASSWORD, DB_PASSWORD, SERVER_SECRET_KEY, CLIENT_SECRET_KEY hanno default committati; un `.env` incompleto parte in silenzio e le sessioni Flask diventano falsificabili (ruoli, is_admin). `session.permanent` mai impostato: `PERMANENT_SESSION_LIFETIME=8h` non ha effetto. Il cookie contiene `api_key` in chiaro (firmato, non cifrato). `/auth/logout` accetta GET. VERIFICATO: `grep -rn permanent src/frontend/flask_app/*.py blueprints` → 0; `grep -rn "change-this\|change_me" src --include=*.py` → solo i default. ORIGINE: revisione fable del 21/09/2026 su ramo `V3.0.0`, commit 8cc5e82. Documento completo: `_output/2026-09-21-TieMeasureFlow-revisione.md` su Wasabi.

Adriano added the bug label 2026-09-21 20:14:52 +00:00

Adriano added the assegnata:jeeg label 2026-09-21 20:23:22 +00:00

Adriano commented

2026-09-21 20:23:23 +00:00

⚡ jeeg — preso in carico 2026-09-21T20:23:23Z
ramo: lavoro/jeeg · capacità: docker, gpu, systemd

⚡ jeeg — **preso in carico** 2026-09-21T20:23:23Z ramo: `lavoro/jeeg` · capacità: docker, gpu, systemd

Adriano commented

2026-09-21 20:23:23 +00:00

ANALISI (passo 1) — ramo lavoro/jeeg, worktree ~/lavoro/TieMeasureFlow su jeeg, lotto A2+B2+C2

MODIFICHE backend (A2): src/backend/config.py — all'avvio, se l'ambiente non è debug/test (usa il flag esistente; se non c'è, env APP_ENV != "dev"/"test") e secret_key o la password DB sono uguali al default committato → RuntimeError con messaggio che nomina la variabile.
MODIFICHE frontend (B2): src/frontend/flask_app/config.py — stessa guardia su SECRET_KEY quando FLASK_DEBUG non è 1 e non TESTING. src/frontend/flask_app/blueprints/auth.py — session.permanent = True al login; /auth/logout solo POST con CSRF (aggiorna il link nei template che lo chiamano: cerca auth.logout in templates/ e trasformalo in form POST).
MODIFICHE deploy (C2): docker-compose.yml — segreti senza default: ${DB_ROOT_PASSWORD:?manca DB_ROOT_PASSWORD} ecc. per DB_ROOT_PASSWORD, DB_PASSWORD, SERVER_SECRET_KEY, CLIENT_SECRET_KEY. docker-compose.dev.yml conserva i default (è sviluppo). .env.example: commento che in produzione sono obbligatori.
TEST: config con default e ambiente produzione → errore; con TESTING/debug → parte. Logout GET → 405.

ANALISI (passo 1) — ramo `lavoro/jeeg`, worktree `~/lavoro/TieMeasureFlow` su jeeg, lotto A2+B2+C2 MODIFICHE backend (A2): src/backend/config.py — all'avvio, se l'ambiente non è debug/test (usa il flag esistente; se non c'è, env APP_ENV != "dev"/"test") e `secret_key` o la password DB sono uguali al default committato → RuntimeError con messaggio che nomina la variabile. MODIFICHE frontend (B2): src/frontend/flask_app/config.py — stessa guardia su SECRET_KEY quando FLASK_DEBUG non è 1 e non TESTING. src/frontend/flask_app/blueprints/auth.py — `session.permanent = True` al login; `/auth/logout` solo POST con CSRF (aggiorna il link nei template che lo chiamano: cerca `auth.logout` in templates/ e trasformalo in form POST). MODIFICHE deploy (C2): docker-compose.yml — segreti senza default: `${DB_ROOT_PASSWORD:?manca DB_ROOT_PASSWORD}` ecc. per DB_ROOT_PASSWORD, DB_PASSWORD, SERVER_SECRET_KEY, CLIENT_SECRET_KEY. docker-compose.dev.yml conserva i default (è sviluppo). .env.example: commento che in produzione sono obbligatori. TEST: config con default e ambiente produzione → errore; con TESTING/debug → parte. Logout GET → 405.