alexferl / zerohttp / 23070180235

94%

Build Type

push

github

Committed by web-flow

Commit Message

fix: SSE code quality improvements and spec compliance (#98)

- Normalize line endings (CRLF/LF/CR) for SSE spec compliance
- Validate event.ID and event.Name to reject CR/LF/NULL characters
- Fix InMemoryReplayer to not hold lock during network I/O
- Remove redundant isOpen() check in Broadcast/BroadcastTo
- Use io.WriteString to eliminate double allocation
- Fix inconsistent error wrapping in SSEWriter methods
- Document Content-Type check heuristic in setupSSEResponse
- Reuse same timestamp in Store for TTL pruning and event
- Return error when Last-Event-ID present but no replayer
- Remove unused isOpen() method

Signed-off-by: alexferl <me@alexferl.com>

Coverage Stats

73 of 79 new or added lines in 1 file covered. (92.41%)

10 existing lines in 3 files now uncovered.

9122 of 9799 relevant lines covered (93.09%)

76.61 hits per line