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:
parent
55e1dd1303
commit
6046d55419
1 changed files with 1 additions and 15 deletions
|
|
@ -15,21 +15,7 @@
|
|||
handle_path /trade-in/api/* {
|
||||
# handle_path вырезает префикс /trade-in перед прокси →
|
||||
# FastAPI получает /api/v1/...
|
||||
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
|
||||
}
|
||||
}
|
||||
reverse_proxy tradein-backend:8000
|
||||
}
|
||||
|
||||
handle /trade-in {
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue