Устойчивость к HTML-ответам: JSON-обработчики ошибок (413/404/500), безопасный разбор ответа на фронтенде
This commit is contained in:
1 parent
7bf6c8df38
commit
e3e850d4b2
2 files changed
+35
-2
No files matched your search
@@ -109,6 +109,24 @@ BASE_URL = os.environ.get("BASE_URL", "https://med.my-dpr.ru").rstrip("/")
|
||||
CORS_ORIGINS = [o.strip() for o in os.environ.get("CORS_ORIGINS", "*").split(",") if o.strip()]
|
||||
|
||||
|
||||
# Обработчики ошибок: всегда возвращаем JSON, чтобы фронтенд
|
||||
# не получал HTML-страницу (иначе response.json() падает с "Unexpected token '<'").
|
||||
@app.errorhandler(413)
|
||||
def too_large(e):
|
||||
return jsonify({"error": "Видео слишком большое (максимум 100 МБ)"}), 413
|
||||
|
||||
|
||||
@app.errorhandler(404)
|
||||
def not_found(e):
|
||||
return jsonify({"error": "Не найдено"}), 404
|
||||
|
||||
|
||||
@app.errorhandler(500)
|
||||
def server_error(e):
|
||||
logger.error(f"Необработанная ошибка: {e}")
|
||||
return jsonify({"error": "Внутренняя ошибка сервера"}), 500
|
||||
|
||||
|
||||
@app.after_request
|
||||
def add_cors_headers(response):
|
||||
origin = request.headers.get("Origin")
|
||||
@@ -470,7 +488,11 @@ def analyze():
|
||||
os.makedirs(work_dir, exist_ok=True)
|
||||
|
||||
video_path = os.path.join(work_dir, f"input{ext}")
|
||||
try:
|
||||
video.save(video_path)
|
||||
except Exception as e:
|
||||
logger.error(f"[{job_id}] Ошибка сохранения видео: {e}")
|
||||
return jsonify({"error": f"Не удалось сохранить видео: {e}"}), 500
|
||||
|
||||
# Сохраняем копию записанного видео (то, что было в кружке) — даже если анализ упадёт
|
||||
saved_video = os.path.join(VIDEO_DIR, f"{job_id}{ext}")
|
||||
|
||||
+13
-2
@@ -483,6 +483,17 @@
|
||||
resultCard.style.display = 'none';
|
||||
setProgress(5, 'Загрузка видео...');
|
||||
|
||||
// Безопасный разбор ответа: если сервер вернул не JSON (например,
|
||||
// HTML-страницу ошибки), не падаем с "Unexpected token '<'".
|
||||
async function parseJson(response) {
|
||||
const ct = (response.headers.get('content-type') || '').toLowerCase();
|
||||
if (ct.includes('application/json')) {
|
||||
return await response.json();
|
||||
}
|
||||
const text = await response.text();
|
||||
return { error: text || `Ошибка сервера (HTTP ${response.status})` };
|
||||
}
|
||||
|
||||
const formData = new FormData();
|
||||
formData.append('video', blob, filename);
|
||||
|
||||
@@ -496,14 +507,14 @@
|
||||
});
|
||||
|
||||
if (!response.ok) {
|
||||
const err = await response.json();
|
||||
const err = await parseJson(response);
|
||||
const e = new Error(err.error || 'Ошибка анализа');
|
||||
e.videoUrl = err.video_url;
|
||||
throw e;
|
||||
}
|
||||
|
||||
setProgress(90, 'Формирование результата...');
|
||||
const data = await response.json();
|
||||
const data = await parseJson(response);
|
||||
|
||||
if (data.error) {
|
||||
showError(data.error);
|
||||
|
||||
Reference in new issue
Block a user