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/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 {
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue