dangernoodle-io / breadboard / 27508436513

100%

push

github

fix(bb_http_client): reconnect on session POST failure + opt-in keep-alive (B1-284) (#428)

- call esp_http_client_close() in bb_http_client_session_post() when
  esp_http_client_perform() fails, so the next perform() opens a fresh
  TCP/TLS connection instead of reusing a wedged dead socket
- add bool keep_alive to bb_http_client_cfg_t; bb_http_client_session_open()
  sets .keep_alive_enable from cfg->keep_alive instead of hardcoded true
- add CONFIG_BB_HTTP_CLIENT_KEEPALIVE (bool, default y) to Kconfig; preserves
  current default behaviour
- session_ensure() in bb_sink_http.c sets cfg.keep_alive from Kconfig via
  #if defined(CONFIG_BB_HTTP_CLIENT_KEEPALIVE), defaulting to true on host
- add static consecutive-transport-failure counter in http_pub_publish(); after
  3 consecutive failures call session_close() so the next publish rebuilds the
  session and TLS from scratch; reset counter to 0 on success
- expose bb_http_client_session_open_count() and
  bb_http_client_session_last_keep_alive() on host mock for test assertions
- add two host tests: session_reset_after_3_consec_failures and
  keep_alive_cfg_threads_into_session_open; all 2088 tests pass, 100% coverage

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

1665 of 1665 branches covered (100.0%)

Branch coverage included in aggregate %.

2775 of 2775 relevant lines covered (100.0%)

1108.29 hits per line