jeffotoni / quick / 363

88%

Build Type

push

circleci

Committed by BRX0002735_prosegur

Commit Message

feat: add responseWriter wrapper to prevent duplicate WriteHeader calls

Implemented a custom responseWriter wrapper to handle multiple
WriteHeader
calls gracefully. The wrapper silently ignores duplicate calls,
preventing
"superfluous response.WriteHeader" errors in edge cases.

Changes:
- Added responseWriter struct with WriteHeader/Write methods in ctx.go
- Modified Reset() in pool.go to automatically wrap http.ResponseWriter
- Updated quick.go to use newCtx() consistently across all Ctx creation
  points
- Removed direct &Ctx{} instantiation in HandlerFunc, mwWrapper, and
  convertQuickToHttpHandler functions

The wrapper prevents crashes when handlers accidentally call response
methods
multiple times while maintaining backward compatibility with existing
code.

Run Details

3 of 47 new or added lines in 4 files covered. (6.38%)

2 existing lines in 1 file now uncovered.

4132 of 8017 relevant lines covered (51.54%)

1351.18 hits per line