alexferl / zerohttp / 23968847330

94%

Build Type

push

github

Committed by web-flow

Commit Message

fix(mediatype): correct DefaultType to set Accept header on request (#165)

* fix(mediatype): correct DefaultType to set Accept header on request

The DefaultType config was incorrectly trying to set the response
Content-Type via a response writer wrapper. This never worked in
practice because renderers (R.JSON, etc.) always set their own
Content-Type headers, overwriting the middleware's value.

Changed DefaultType to instead set the Accept header on the request
when the client sends an empty Accept header or */*, allowing
handlers to perform proper content negotiation.

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

* fix(mediatype): correct DefaultType, wildcard matching, and validation

- DefaultType now sets request Accept header (not response Content-Type)
- Add bidirectional wildcard matching: client `type/*` matches server types
- Fix chunked body validation: check ContentLength != 0 (not > 0)
- Fix suffix wildcards: `+*` patterns require a suffix to match
- Make suffix matching symmetric: both sides must agree
- Remove redundant hasSuffix checks

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

---------

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

Coverage Stats

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

4 existing lines in 2 files now uncovered.

10537 of 11217 relevant lines covered (93.94%)

434.12 hits per line