lekss361/gendesign
1
0
Fork 0
Code Issues 57 Pull requests Projects Releases Packages Wiki Activity Actions

fix(frontend): apiFetchWithStatus body stream double-read crash

Browse source 
Per user report: 'Failed to execute text on Response: body stream already read'
при analyze запросе с error response.

## Root cause

apiFetchWithStatus делал:
1. await response.json() (consumes body stream даже на parse error)
2. catch → await response.text() ← FAILS — stream already consumed

fetch API: response body — ReadableStream, consumable один раз.

## Fix

Read text() once → attempt JSON.parse. Если parse failed (HTML error page
от Caddy / 502 / etc) — оборачиваем в {detail: rawText}. Никаких
double-reads.

## Impact

Любой non-JSON error response (502/504/HTML/etc) теперь показывает реальный
detail вместо misleading 'body stream' error.

Refs: user report 2026-05-14
This commit is contained in:
lekss361 2026-05-15 07:34:37 +03:00
parent c6cabc1990
commit e109400fcd
1 changed files with 10 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
frontend/src/lib/api.ts
View file

@ -47,13 +47,17 @@ export async function apiFetchWithStatus<T>(
},
});
let body: unknown = null;
// 204 No Content имеет пустое тело; для всех остальных пытаемся JSON
// 204 No Content имеет пустое тело; для всех остальных читаем как text один раз
// (fetch body stream consumable только один раз — нельзя сначала .json() потом .text()).
if (response.status !== 204) {
try {
body = await response.json();
} catch {
// fallback на text() — может прилететь HTML error page от Caddy
body = { detail: await response.text() };
const rawText = await response.text();
if (rawText) {
try {
body = JSON.parse(rawText);
} catch {
// Не JSON (HTML error page от Caddy / 502 / etc) — оборачиваем в detail
body = { detail: rawText };
}
}
}
if (!response.ok) {