dangernoodle-io / breadboard / 29624074236

96%

push

github

feat: add bb_http_client session health snapshot (#915)

- health attaches to the keep-alive SESSION handle only -- the one-shot
  get/post calls have no persistent object to hang health on, so they are
  documented as out of scope
- add connected/last_ok_ms/fail_count/tls_error_code to each session's
  health state (host + espidf backends)
- fail_count bumps on a transport failure OR a 5xx response; a 4xx is a
  valid server response, not a transport failure, so it leaves fail_count
  untouched
- tls_error_code is set whenever a session_post result carries a non-zero
  value and otherwise left at its last-reported value (never reset to 0 on
  a later success)
- add bb_http_client_session_health_snap_t + bb_http_client_session_health_desc
  (bb_serialize descriptor) + bb_http_client_session_health_fill, mirroring
  bb_tcp_client's / bb_mqtt_client's per-instance health wire surface
- neither session struct carried an existing lock, and a diag/egress
  consumer reading health from a different task than the one driving
  session_post/session_close is a real design goal, so each backend embeds
  a pthread_mutex-guarded health sub-struct (mirrors bb_tcp_client's
  per-instance health coherency)
- bb_http_client_session_close now clears connected without bumping
  fail_count (a caller-initiated close is not a transport failure)
- add deterministic host test hooks (mock clock + the existing session mock
  knobs) and cover every new branch

7135 of 7770 branches covered (91.83%)

Branch coverage included in aggregate %.

54 of 54 new or added lines in 2 files covered. (100.0%)

12250 of 12467 relevant lines covered (98.26%)

2327.13 hits per line