You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
🚩 Other SSE endpoint lacks keepalive and proxy headers
The chat completions SSE endpoint at src/web/openai.rs:380 uses Sse::new(stream).into_response() without any keepalive or x-accel-buffering: no header. If the motivation for this PR is that sparse SSE streams get dropped by proxies/load balancers, the same issue could affect the other SSE endpoint. Consider whether the responses_sse_response helper should be reused there as well.
Was this helpful? React with 👍 or 👎 to provide feedback.
🚩 Other SSE endpoint lacks keepalive and proxy headers
The chat completions SSE endpoint at
src/web/openai.rs:380usesSse::new(stream).into_response()without any keepalive orx-accel-buffering: noheader. If the motivation for this PR is that sparse SSE streams get dropped by proxies/load balancers, the same issue could affect the other SSE endpoint. Consider whether theresponses_sse_responsehelper should be reused there as well.Was this helpful? React with 👍 or 👎 to provide feedback.
Originally posted by @devin-ai-integration[bot] in #175 (comment)