Adriano/TieMeasureFlow
1
0
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

Visione: file non immagine o grafo malformato → 500 del worker, 502 senza motivo al Maker #15

New Issue
Closed
opened 2026-09-21 20:14:42 +00:00 by Adriano · 3 comments
Adriano commented 2026-09-21 20:14:42 +00:00
Owner
Copy Link
Copy Source

GRAVITÀ: alta
FILE: src/vision_worker/main.py:38-48 ; src/backend/api/routers/vision.py:49 ; src/backend/api/routers/tasks.py:182
COSA: Image.open sta fuori dal try; è gestito solo ValueError. UnidentifiedImageError, e KeyError/AttributeError/TypeError di Task.from_dict, diventano 500. Il backend controlla solo l'header content_type; vision_json non è validato alla creazione del task.
INPUT: (a) file di testo dichiarato image/png; (b) graph="[]"; (c) strumento senza "kind". Tutti → 500 worker → 502 «refused the job: Internal Server Error». Dovrebbe essere 422 con motivo.
VERIFICATO: lettura di main.py:24-48; probe TestClient del revisore (Python 3.13) → A, B, C = 500.

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À: alta FILE: src/vision_worker/main.py:38-48 ; src/backend/api/routers/vision.py:49 ; src/backend/api/routers/tasks.py:182 COSA: `Image.open` sta fuori dal try; è gestito solo `ValueError`. `UnidentifiedImageError`, e `KeyError`/`AttributeError`/`TypeError` di `Task.from_dict`, diventano 500. Il backend controlla solo l'header content_type; `vision_json` non è validato alla creazione del task. INPUT: (a) file di testo dichiarato image/png; (b) graph="[]"; (c) strumento senza "kind". Tutti → 500 worker → 502 «refused the job: Internal Server Error». Dovrebbe essere 422 con motivo. VERIFICATO: lettura di main.py:24-48; probe TestClient del revisore (Python 3.13) → A, B, C = 500. 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:42 +00:00
Adriano added the assegnata:jeeg label 2026-09-21 20:22:31 +00:00
Adriano commented 2026-09-21 20:22:33 +00:00
Author
Owner
Copy Link
Copy Source

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

⚡ jeeg — **preso in carico** 2026-09-21T20:22:32Z ramo: `lavoro/jeeg` · capacità: docker, gpu, systemd
Adriano commented 2026-09-21 20:22:33 +00:00
Author
Owner
Copy Link
Copy Source

ANALISI (passo 1) — ramo lavoro/jeeg, worktree ~/lavoro/TieMeasureFlow su jeeg, lotto C1

MODIFICHE: src/vision_worker/main.py — Image.open(...).convert dentro try: UnidentifiedImageError/OSError → 422 «image is not decodable»; attorno a run_graph: KeyError, TypeError, AttributeError, ValueError provenienti dalla costruzione del grafo → 422 «malformed graph: : ». Distingui costruzione ed esecuzione se src/vision/runner.py lo permette (due fasi: build → 422, run → 500 con motivo nel detail). Il grafo deve essere un dict: lista/altro → 422. src/backend/services/vision_service.py: un 422 del worker si propaga come 422 con il detail del worker (non 502). src/backend/api/routers/vision.py:49: verifica i magic bytes dell'immagine (PNG/JPEG/BMP/TIFF/WebP/GIF) → 415/422 prima di chiamare il worker.
TEST: src/vision_worker/tests/test_worker.py (Python 3.13, uv run --python 3.13 --extra vision-worker --extra dev pytest src/vision src/vision_worker): file di testo → 422; graph [] → 422; strumento senza kind → 422. Base: scratchpad/probe_worker.py.
NON TOCCARE: vendor/visionsuite; src/backend/api/routers/tasks.py.

ANALISI (passo 1) — ramo `lavoro/jeeg`, worktree `~/lavoro/TieMeasureFlow` su jeeg, lotto C1 MODIFICHE: src/vision_worker/main.py — `Image.open(...).convert` dentro try: `UnidentifiedImageError`/`OSError` → 422 «image is not decodable»; attorno a `run_graph`: `KeyError`, `TypeError`, `AttributeError`, `ValueError` provenienti dalla costruzione del grafo → 422 «malformed graph: <tipo>: <msg>». Distingui costruzione ed esecuzione se `src/vision/runner.py` lo permette (due fasi: build → 422, run → 500 con motivo nel detail). Il grafo deve essere un dict: lista/altro → 422. src/backend/services/vision_service.py: un 422 del worker si propaga come 422 con il detail del worker (non 502). src/backend/api/routers/vision.py:49: verifica i magic bytes dell'immagine (PNG/JPEG/BMP/TIFF/WebP/GIF) → 415/422 prima di chiamare il worker. TEST: src/vision_worker/tests/test_worker.py (Python 3.13, `uv run --python 3.13 --extra vision-worker --extra dev pytest src/vision src/vision_worker`): file di testo → 422; graph `[]` → 422; strumento senza `kind` → 422. Base: scratchpad/probe_worker.py. NON TOCCARE: vendor/visionsuite; src/backend/api/routers/tasks.py.
Adriano commented 2026-09-21 21:56:18 +00:00
Author
Owner
Copy Link
Copy Source

Soluzione

ESITO: fatto
COSA: Worker: immagine non decodificabile, grafo non-oggetto o malformato (GraphBuildError) → 422 con motivo; errore in esecuzione → 500 con tipo e messaggio. Il backend inoltra 422/503/504 del worker e verifica i magic bytes dell'immagine prima di chiamarlo.
COMMIT (ramo lavoro/jeeg, unito su V3.0.0 a d483fa1): 9afe1e7
PROVA: uv run --extra server --extra client --extra dev pytest -q → 622 passed, 4 skipped (erano 391); visione su Python 3.13 → 16 passed (erano 8).

**Soluzione** ESITO: fatto COSA: Worker: immagine non decodificabile, grafo non-oggetto o malformato (`GraphBuildError`) → 422 con motivo; errore in esecuzione → 500 con tipo e messaggio. Il backend inoltra 422/503/504 del worker e verifica i magic bytes dell'immagine prima di chiamarlo. COMMIT (ramo lavoro/jeeg, unito su V3.0.0 a d483fa1): 9afe1e7 PROVA: `uv run --extra server --extra client --extra dev pytest -q` → 622 passed, 4 skipped (erano 391); visione su Python 3.13 → 16 passed (erano 8).
Adriano removed the assegnata:jeeg label 2026-09-21 21:56:18 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
assegnata:daitarn
assegnata:jeeg
attesa:adriano
attesa:cliente
attesa:macchina
bug
Difetto trovato in revisione o in uso
 sviluppo
Sviluppo o miglioria da fare
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Adriano
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Adriano/TieMeasureFlow#15
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?