dangernoodle-io / breadboard / 25137450350

100%

push

github

fix(bb_http): clamp max_open_sockets to LWIP_MAX_SOCKETS-3 (#155)

esp-idf httpd reserves 3 sockets internally and asserts that
max_open_sockets <= CONFIG_LWIP_MAX_SOCKETS - 3 at startup. if the
configured value exceeds this cap, httpd_start aborts with "Config
option max_open_sockets is too large", causing a prov-mode bootloop.

this was triggered in TaipanMiner v0.16.0/v0.16.1 when LWIP_MAX_SOCKETS
was reduced to 8 on the tdongle-s3 target: the default max_open_sockets
of 9 exceeds the effective cap of 5, crashing every provisioning attempt.

the clamp is applied unconditionally at runtime so no consumer sdkconfig
changes are required. a bb_log_w warning is emitted when the value is
reduced so consumers can spot accidental over-tuning in their logs.

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

336 of 336 branches covered (100.0%)

Branch coverage included in aggregate %.

527 of 527 relevant lines covered (100.0%)

487.36 hits per line