audio streaming app plyr.fm
38
fork

Configure Feed

Select the types of activity you want to include in your feed.

fix: short-circuit logfire proxy when browser_observability is disabled (#1289)

the frontend toggle from #1288 only prevents new page loads from
calling initObservability(). stale cached clients (Cloudflare Pages)
continue hammering POST /logfire-proxy — 3,458 requests in 24 minutes
averaging 1.9s each, saturating the threadpool and causing /tracks/top
to take 10-18s.

guard the backend endpoint directly: return 204 immediately when the
flag is off, so no stale client can reach logfire_proxy().

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

authored by

nate nowack
Claude Opus 4.6 (1M context)
and committed by
GitHub
1f1bcc7c e7847684

+2
+2
backend/src/backend/api/meta.py
··· 188 188 stays server-side. protected by CORS (allowed origins only) and 189 189 global rate limiting. 190 190 """ 191 + if not settings.app.browser_observability: 192 + return PlainTextResponse("browser observability disabled", status_code=204) 191 193 return await logfire_proxy(request)