dangernoodle-io / breadboard / 25064061165

100%

push

github

fix(bb_http): correct set_status lifetime and dedup Content-Type via set_type (#121)

bb_http_resp_set_status formatted the status code into a stack buffer; httpd_resp_set_status only stores the pointer, so the status string was already gone by the time the response was sent — wire showed an empty status (HTTP/1.1 \r\n). Map known codes to static literals so the storage outlives the call.

Add bb_http_resp_set_type wrapper around httpd_resp_set_type and migrate every callsite from set_header(req, "Content-Type", ...) to set_type(req, ...). Without set_type the httpd auto-emits a default Content-Type: text/html and any explicit Content-Type via set_hdr is appended, producing duplicates on the wire. Verified on hardware before/after.

272 of 272 branches covered (100.0%)

Branch coverage included in aggregate %.

431 of 431 relevant lines covered (100.0%)

146.06 hits per line