chore(tradein): drop shared Caddyfile timeout from #654 — defer to devops

Review-bot flagged Caddyfile.tradein-fragment as shared gendsgn.ru gateway
infra needing devops review. The app-level asyncio.wait_for budgets already
bound the slow enrichment chain, so the 502 fix stands without the gateway
timeout. Caddy read/write timeout to be applied separately under devops review.
This commit is contained in:
lekss361 2026-05-29 20:07:04 +03:00
parent 55e1dd1303
commit 6046d55419

View file

@ -15,21 +15,7 @@
handle_path /trade-in/api/* { handle_path /trade-in/api/* {
# handle_path вырезает префикс /trade-in перед прокси → # handle_path вырезает префикс /trade-in перед прокси →
# FastAPI получает /api/v1/... # FastAPI получает /api/v1/...
reverse_proxy tradein-backend:8000 { reverse_proxy tradein-backend:8000
# #654: явный read/write timeout. POST /api/v1/trade-in/estimate делает
# цепочку последовательных внешних вызовов (geocode → Overpass → Yandex
# valuation → IMV → Cian). In-app time-budget (config.py
# estimate_*_timeout_s) суммарно ≈28s worst-case; ставим upstream timeout
# ВЫШЕ него (45s), чтобы 503-деградация из приложения доходила до клиента
# как 503, а НЕ обрезалась прокси в непрозрачный 502/504. Делает поведение
# детерминированным и согласованным с in-app budget.
# SHARED-INFRA: этот фрагмент инклудится в основной gendsgn.ru Caddy-блок
# на bot-server — требует review devops перед deploy.
transport http {
read_timeout 45s
write_timeout 45s
}
}
} }
handle /trade-in { handle /trade-in {