raystack / frontier / 31673246417

48%

main: 49%

Build Type

Pull #1877

github

Committed by whoAbhishekSah

Commit Message

fix(server): abort committed responses instead of appending an error body

When a UI handler panics after writing part of a response, the status
line is already on the wire: http.Error cannot change it and would only
append the error text to the partial body, so the client would read a
corrupt 200. Track response commitment; once committed, log the panic
and re-panic with http.ErrAbortHandler so net/http drops the connection
and the client sees a truncated response instead of a fake success. The
wrapper forwards Flush for the reverse proxy's streaming.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Pull Request Pull Request #1877: fix(server): recover from panics in RPC and UI HTTP handlers

Coverage Stats

41 of 42 new or added lines in 2 files covered. (97.62%)

13 existing lines in 1 file now uncovered.

19232 of 39851 relevant lines covered (48.26%)

15.4 hits per line