stacklok / toolhive / 30264573849

push

github

Audit stream opens on flush and pre-write panics (#6005)

Stacked on #5874. Two fast-follows from rdimitrov's review:

- Flush() now logs the stream connection event: a handler that
  establishes the stream by flushing headers and then blocks never hits
  WriteHeader/Write, so the event was delayed until the first data
  write or lost entirely.
- The stream-open log moved into a defer registered before ServeHTTP,
  so a panic in an inner handler still produces the event during
  unwinding. On chains whose recovery middleware runs outside audit
  (the vMCP Serve path) the event previously vanished.

7 of 7 new or added lines in 1 file covered. (100.0%)

80219 of 115339 relevant lines covered (69.55%)

82.84 hits per line