dangernoodle-io / breadboard / 27456836199

100%

push

github

feat(bb_sink_http): structured per-header config w/ delimited NVS storage + per-header secret masking + generic bb_http_client_session_set_header (#412)

- add bb_http_client_session_set_header(session, name, value) to bb_http_client API; host mock records all applied headers (name→value) per session for test assertion via bb_http_client_session_find_header
- add BB_SINK_HTTP_HEADERS_MAX (8), BB_SINK_HTTP_HEADER_NAME_MAX (64), BB_SINK_HTTP_HEADER_VALUE_MAX (256), BB_SINK_HTTP_CLIENT_ID_MAX (64) to bb_sink_http.h
- add bb_sink_http_header_t { name, value, secret } and client_id + headers/num_headers fields to bb_sink_http_cfg_t
- persist headers as one delimited string in NVS key "headers" (format: [*]name: value per \n-separated line; '*' prefix marks secret)
- pure host-testable fns: bb_sink_http_parse_headers, bb_sink_http_serialize_headers, bb_sink_http_merge_headers, bb_sink_http_header_name_valid, bb_sink_http_header_value_valid
- injection guard: names reject ':', whitespace, control chars; values reject '\r'/'\n'
- apply X-Client-Id (client_id or bb_nv_config_hostname fallback) + all headers via session_set_header at session open; re-apply after set_cfg
- GET "http" telemetry section emits client_id (clear) + headers array: secret rows omit value, include set:true; non-secret rows include value
- PATCH "http" section: client_id persisted; headers array merged (secret+blank → preserve existing value by name; omitted names removed)
- 39 new tests across test_bb_sink_http.c and test_bb_sink_http_telemetry.c; 1977 total tests pass clean build

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

1665 of 1665 branches covered (100.0%)

Branch coverage included in aggregate %.

2775 of 2775 relevant lines covered (100.0%)

1075.32 hits per line